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

Replace ESLint with StandardJS #898

Merged
merged 4 commits into from Jul 25, 2022
Merged

Conversation

Jozzey
Copy link
Contributor

@Jozzey Jozzey commented Jul 22, 2022

DEFRA/water-abstraction-team#9

Defra's JavaScript standard is to use StandardJS for linting. We currently use ESLint, configured to Semi-Standard (ie. StandardJS with semicolons). For consistency, we should change this to the regular Standard rules, and replace ESLint with StandardJS while we're at it.

DEFRA/water-abstraction-team#9

Defra's JavaScript standard is to use StandardJS for linting. We currently use ESLint, configured to Semi-Standard (ie. StandardJS with semicolons). For consistency, we should change this to the regular Standard rules, and replace ESLint with StandardJS while we're at it.
@Jozzey Jozzey added the housekeeping Refactoring, tidying up or other work which supports the project label Jul 22, 2022
@Jozzey Jozzey self-assigned this Jul 22, 2022
@Jozzey Jozzey marked this pull request as ready for review July 22, 2022 17:07
@Jozzey
Copy link
Contributor Author

Jozzey commented Jul 22, 2022

Sonarcloud is complaining because the unit test coverage is only 89.9%. Since I've just been replacing eslint with standard I'm not going to be sorting that out in this PR.

@Jozzey Jozzey requested a review from StuAA78 July 22, 2022 17:10
@Jozzey Jozzey merged commit 9c93884 into develop Jul 25, 2022
@Jozzey Jozzey deleted the replace-eslint-with-standardjs branch July 25, 2022 09:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
housekeeping Refactoring, tidying up or other work which supports the project
Projects
None yet
2 participants