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 story cards without encounter_code #1080

Closed
wants to merge 13 commits into from
Closed

Fix story cards without encounter_code #1080

wants to merge 13 commits into from

Conversation

boutchitos
Copy link
Contributor

I fix validator.py :

  • It validates a story card has an encounter_code
  • It fix (with --fix_formatting) story card without encounter_code by setting it from the previous scanned cards.

Related to issues:
#1075
#983 (encounter_code only, I did not fix the images).

Needs this issue/PR approuved to get validation of encounter cards: #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.
Upgrades:

validate.py is green. Schema more relax, some cards fixed directly.
will be used to auto-fix errors.
@boutchitos boutchitos closed this Jan 2, 2022
@boutchitos
Copy link
Contributor Author

Close-it :) not good at this yet, first time working with a fork. This PR contains too much files.

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