Skip to content

Raise exception if validator mutates instance#1339

Closed
stephenfin wants to merge 2 commits intopython-jsonschema:mainfrom
stephenfin:mutating-validators
Closed

Raise exception if validator mutates instance#1339
stephenfin wants to merge 2 commits intopython-jsonschema:mainfrom
stephenfin:mutating-validators

Conversation

@stephenfin
Copy link
Copy Markdown

@stephenfin stephenfin commented Mar 5, 2025

There are test failures which I have yet to investigate in depth. In addition, this may not be the direction we want to go (if there is a direction). Hopefully it can kickstart the conversation though.

Closes: #1338


📚 Documentation preview 📚: https://python-jsonschema--1339.org.readthedocs.build/en/1339/

stephenfin and others added 2 commits March 5, 2025 15:21
@Julian
Copy link
Copy Markdown
Member

Julian commented Mar 26, 2025

Hey! Thanks for raising. Closing as per #1338 (comment) but still appreciated and willing to continue discussing there!

@Julian Julian closed this Mar 26, 2025
@stephenfin stephenfin deleted the mutating-validators branch March 27, 2025 16:22
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.

Mutating validators are difficult to debug

2 participants