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

Update GitHub super linter #199

Merged
merged 4 commits into from
Jan 3, 2022
Merged

Conversation

kylelaker
Copy link
Contributor

Newer versions of the super linter changed to using eslint for JSON
linting; however, our eslintrc doesn't specify the JSON plugin. This
adds it as a dependency and to the configuration file to ensure that the
plugin is available and JSON can be parsed correctly.

@kylelaker kylelaker force-pushed the feature/update-github-super-linter branch 2 times, most recently from 0f85883 to e0dfa67 Compare December 21, 2021 17:29
@kylelaker kylelaker added dependencies Pull requests that update a dependency file github_actions Pull requests that update Github_actions code javascript Pull requests that update Javascript code labels Dec 21, 2021
@kylelaker kylelaker marked this pull request as ready for review December 21, 2021 19:57
Newer versions of the super linter changed to using eslint for JSON
linting; however, our eslintrc doesn't specify the JSON plugin. This
adds it as a dependency and to the configuration file to ensure that the
plugin is available and JSON can be parsed correctly.
This was identified by `eslint` as something that should be a production
rather than dev dependencies. This is due to it being imported and
executed directly via `index.js`.
This resolves linter errors that were identified after fixing the
plugin.
This ensures that the example app is captured correctly in the eslint
config and that the app complies with the linting requirements.
@kylelaker kylelaker force-pushed the feature/update-github-super-linter branch from 047dc02 to a3a423e Compare December 29, 2021 18:31
@hreineck hreineck merged commit ff104a4 into develop Jan 3, 2022
@rgauss rgauss deleted the feature/update-github-super-linter branch January 11, 2022 12:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file github_actions Pull requests that update Github_actions code javascript Pull requests that update Javascript code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants