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 validate_card is not called for encounter cards in validate.py #1078

Closed
wants to merge 5 commits into from
Closed

Fix validate_card is not called for encounter cards in validate.py #1078

wants to merge 5 commits into from

Conversation

boutchitos
Copy link
Contributor

Issue: #1077

May be related to version's packages, but just
running the command: `python validate.py --fix_formatting`
modifies all the json files.

This is the results:
- mostly whitespaces
- attributes ordering

My packages :
$ pip list
Package    Version
---------- -------
attrs      21.4.0
jsonschema 4.3.2
pip        21.3.1
pyrsistent 0.18.0
setuptools 57.4.0
Ran:
yarn install
yarn prettier
yarn prettier:check
Fix/validate formats all json

Workflow is green!
Implied fields are now logged.
Also fix cards directly removing like sanity: null.
@boutchitos boutchitos closed this Jan 1, 2022
@boutchitos
Copy link
Contributor Author

I pushed too much on the branch's PR...

@boutchitos boutchitos deleted the fix/1077-validate_encounter_cards branch January 1, 2022 22:36
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.

1 participant