Skip to content

Feat(package.json): add eslint-staged scripts

Michael Lunøe requested to merge mlunoe-add-eslint-staged-scripts into master

What does this MR do?

This MR adds optional eslint-staged and eslint-staged-fix scripts to the package.json, so that developers can add these to run in their pre-commit hooks. The main reason to add these scripts is for developers, who don't want to run a linter in their editor and run prettier on save, but rather have it as a pre-commit hook.

An example pre-commit script could look like this:

#!/bin/sh
set -e

yarn prettier-staged
yarn eslint-staged
yarn stylelint

Screenshots

Screenshot_2020-06-09_at_14.23.45

Does this MR meet the acceptance criteria?

Conformity

Availability and Testing

This MR introduces little risk as these scripts have to be manually used for them to be active. It could be that some files are included erroneously, but even when added as a pre-commit hook, this can always be overridden with --no-verify and finally the build would still be the final source of truth if the MR should pass.

Security

If this MR contains changes to processing or storing of credentials or tokens, authorization and authentication methods and other items described in the security review guidelines:

  • [-] Label as security and @ mention @gitlab-com/gl-security/appsec
  • [-] The MR includes necessary changes to maintain consistency between UI, API, email, or other methods
  • [-] Security reports checked/validated by a reviewer from the AppSec team
Edited by Michael Lunøe

Merge request reports