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

tools: disable schema validation #41300

Merged

Conversation

naomi-lgbt
Copy link
Member

Checklist:

  • I have read freeCodeCamp's contribution guidelines.
  • My pull request has a descriptive title (not a vague title like Update index.md)
  • My pull request targets the main branch of freeCodeCamp.
  • All the files I changed are in the same world language, for example: only English changes, or only Chinese changes, etc.

Closes #XXXXX

Changes the schema validation script to print warnings to the
console instead of throwing errors. This will no longer cause CI
to fail on missing/extra/empty translation object keys.

After merge, will need to re-run CI on #41239 (potentially with a rebase).

Changes the schema validation script to print warnings to the
console instead of throwing errors. This will no longer cause CI
to fail on missing/extra/empty translation object keys.
@naomi-lgbt naomi-lgbt requested a review from a team February 28, 2021 15:45
@gitpod-io
Copy link

gitpod-io bot commented Feb 28, 2021

@naomi-lgbt naomi-lgbt added the scope: tools/scripts Scripts for supporting dev work, generating config and build artifacts, etc. label Feb 28, 2021
@raisedadead raisedadead merged commit eda444a into freeCodeCamp:main Feb 28, 2021
@naomi-lgbt naomi-lgbt deleted the tools/disable-schema-validation branch March 5, 2021 04:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
scope: tools/scripts Scripts for supporting dev work, generating config and build artifacts, etc.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants