-
Notifications
You must be signed in to change notification settings - Fork 62
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
issue267: basic check for removal of schemas #270
Conversation
To make reviews easier I will leave this PR for maintainers consideration. |
Fixed localization issue above by running |
@reuvenharrison Thank you so much! |
Fix for #267
This PR adds a new optional check that allows us to fail when a model is removed.
Testing
For some reason, localisation is not picked up:
List of changes
Note: please let me know if there is any further tests I can run to validate my PR.