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
Stop modifying behavior of yaml on load #1826
Conversation
Only modify a copy of ``SafeLoader`` to ensure that we don't change the behaviour of ``pyyaml`` for everyone.
for more information, see https://pre-commit.ci
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #1826 +/- ##
=======================================
Coverage ? 98.98%
=======================================
Files ? 37
Lines ? 4153
Branches ? 967
=======================================
Hits ? 4111
Misses ? 26
Partials ? 16
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
CodSpeed Performance ReportMerging #1826 will not alter performanceComparing Summary
|
Try to address the CodeQL warning.
for more information, see https://pre-commit.ci
Note that CodeQL is happy now - the failing tests seem to be down to problems with the upload of coverage because the github workflow is missing a token to upload to codecov.io - likely unrelated to the content of the PR, as nothing in that direction has changed. |
…t the codecov isue is resolved.
@gluap
OK, I will fix it to merge the PR. |
Only modify a copy of
SafeLoader
to ensure that we don't change the behaviour ofpyyaml
for everyone.Fixes #1825