Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix root classpath resolve #1000

Merged
merged 1 commit into from
Mar 28, 2024
Merged

Conversation

justin-tay
Copy link
Contributor

Fixes the IRI resolve at the root when there is no starting slash.

For instance given classpath:resource.json and resolving test.json will now return classpath:test.json.

@codecov-commenter
Copy link

codecov-commenter commented Mar 28, 2024

Codecov Report

Attention: Patch coverage is 76.92308% with 3 lines in your changes are missing coverage. Please review.

Project coverage is 78.79%. Comparing base (7abb68c) to head (4e0bd48).

Files Patch % Lines
...rc/main/java/com/networknt/schema/AbsoluteIri.java 75.00% 1 Missing and 2 partials ⚠️

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files
@@             Coverage Diff              @@
##             master    #1000      +/-   ##
============================================
+ Coverage     78.73%   78.79%   +0.06%     
- Complexity     1955     1959       +4     
============================================
  Files           172      172              
  Lines          6334     6344      +10     
  Branches       1247     1252       +5     
============================================
+ Hits           4987     4999      +12     
+ Misses          881      876       -5     
- Partials        466      469       +3     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@stevehu stevehu merged commit 61323ba into networknt:master Mar 28, 2024
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants