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 linting through ESLint using AirBnB style guide #7

Open
wants to merge 11 commits into
base: main
Choose a base branch
from

Conversation

fschrempf
Copy link

@fschrempf fschrempf commented Nov 13, 2022

This PR adds linting using ESLint and the AirBnB style guide. It also applies the automatic fixes created by ESLint. The remaining issues need to be resolved manually.

@AlexRoehm If you generally agree with using linting and also with the chosen style guide, you could merge this and we can start fixing the remaining issues manually until the GitHub CI shows a green tick. If you have other ideas or disagree, just let me know. Thanks!

@fschrempf fschrempf force-pushed the add-linting branch 2 times, most recently from 5aacd75 to 950e019 Compare November 13, 2022 14:09
fschrempf and others added 8 commits December 16, 2022 15:56
config.example.yml: Fix indentation and comments
* Remove matrix as long as only one build config is needed
* Upgrade checkout and setup-node actions to v3
* Use 'npm ci' instead of 'npm install'
This allows to get CI test results even on feature branches, etc.
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