HomePhabricator
Diffusion Comm 0409c049bfaa

[RN72] Fix lint-staged

Description

[RN72] Fix lint-staged

Summary:
This finally resolves ENG-1011. It's necessary because ESLint 8 has removed the CLIEngine class.

I got this logic from the lint-staged README.md file.

Depends on D10543

Test Plan:

  1. Edit a file that is in .eslintignore but not .gitignore. I opted for some file in native/flow-typed
  2. Add a line that isn't formatted in a way that Prettier likes
  3. Confirm that lint-staged doesn't fix the file on commit
  4. Remove native/flow-typed from .eslintignore and try again. Confirm that this time, lint-staged fixes the file on commit

Reviewers: atul

Reviewed By: atul

Subscribers: tomek

Differential Revision: https://phab.comm.dev/D10544

Details

Provenance
ashoatAuthored on Jan 4 2024, 10:11 AM
Reviewer
atul
Differential Revision
D10544: [RN72] Fix lint-staged
Parents
rCOMM32100dd2534e: [RN72][skip-ci] Introduce --report-unused-disable-directives to ESLint…
Branches
Unknown
Tags
Unknown