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

Add option to print config #208

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

MatthewRines
Copy link

New option to just print the configuration for feedback to the user.
If the config is not valid, the runner does what it did before and
exits with 2 after displaying the errors.

When the option is set,the linter prints the options, rules, and
paths it would use and exits with a 0 status.

New option to just print the configuration for feedback to the user.
If the config is not valid, the runner does what it did before and
exits with 2 after displaying the errors.

When the option is set,the linter prints the options, rules, and
paths it would use and exits with a 0 status.
@MatthewRines MatthewRines requested a review from cjoudrey January 21, 2020 00:01
@cjoudrey
Copy link
Owner

Hey @MatthewRines! Thanks for the pull request! Really appreciate the help. 😄

Out of curiosity, what you made want to add this option? Were you trying to debug something about your config?

@MatthewRines
Copy link
Author

Hey no problem!

I am working on an editor integration with emacs via flycheck so the options may be programmatically decided by the user's config. I wanted an option that could just verify the config and exit so if there is a problem somewhere we can eliminate graphql-schema-linter as the issue.

@cjoudrey
Copy link
Owner

Just realized I left this PR hanging so wanted to circle back to it. Was this still something you needed?

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.

2 participants