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

Neutralize Schema Linter CI workflow #1723

Merged
merged 7 commits into from Feb 22, 2021
Merged

Neutralize Schema Linter CI workflow #1723

merged 7 commits into from Feb 22, 2021

Conversation

szepeviktor
Copy link
Contributor

@szepeviktor szepeviktor commented Feb 18, 2021

Long-standing issue goes to sleep 🛌

While #1342 is being worked on...

@szepeviktor
Copy link
Contributor Author

Done 🍏

Runs and fails silently.

✖ 441 errors detected

@szepeviktor szepeviktor changed the title Neutralize schema linter CI workflow Neutralize Schema Linter CI workflow Feb 18, 2021
@jasonbahl
Copy link
Collaborator

@szepeviktor thanks for working on this! 🙌

@jasonbahl jasonbahl merged commit 7538a98 into wp-graphql:develop Feb 22, 2021
@szepeviktor szepeviktor deleted the patch-2 branch February 22, 2021 22:04
@szepeviktor
Copy link
Contributor Author

szepeviktor commented Feb 22, 2021

✖ 441 errors detected

Do you know a junior contributor who will clean up the schema?

@sol-idsnake
Copy link

@szepeviktor I don't know if I qualify, but I have been looking to get my feet wet with contributions to the project for a while. Let me know if I can help.

@szepeviktor
Copy link
Contributor Author

@sol-idsnake Your contribution is very welcome.
I suggest you to categorize those 441 schema errors and start with rare ones.
Maybe we could ignore missing description errors...

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.

None yet

3 participants