-
Notifications
You must be signed in to change notification settings - Fork 308
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 calculation of schema location #1239
Fix calculation of schema location #1239
Conversation
Thanks for submitting your first pull request! You are awesome! 🤗 |
Codecov ReportPatch coverage:
Additional details and impacted files@@ Coverage Diff @@
## main #1239 +/- ##
==========================================
- Coverage 80.39% 80.27% -0.13%
==========================================
Files 68 68
Lines 8263 8263
Branches 1600 1600
==========================================
- Hits 6643 6633 -10
- Misses 1198 1206 +8
- Partials 422 424 +2
... and 3 files with indirect coverage changes Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report in Codecov by Sentry. |
Co-authored-by: Zachary Sailer <[email protected]>
Hehe, so, the trailing slash is needed to properly calculate the relative location of the schema. I tried to add it to the definition of the path const but it seemed to break some test cases, I can play more with this later today in case you guys don't like the current approach. |
Nah, I think this is good to go. Thank you @lresende. :) |
Move away of using lstrip to manipulate the base URI while calculating schema location
Fixes #1238