From 7721e2be9d7cd6695a16385f3a8965625da162c4 Mon Sep 17 00:00:00 2001 From: Zach Bloomquist Date: Wed, 4 Jan 2023 10:39:55 -0500 Subject: [PATCH] chore: remove lint-changed from scripts/docs (#25308) --- CONTRIBUTING.md | 10 ++-------- package.json | 1 - 2 files changed, 2 insertions(+), 9 deletions(-) diff --git a/CONTRIBUTING.md b/CONTRIBUTING.md index 7822e0295eb4..dc8904739c1c 100644 --- a/CONTRIBUTING.md +++ b/CONTRIBUTING.md @@ -335,14 +335,8 @@ Many Cypress packages print out debugging information to console via the `debug` We use [eslint](https://eslint.org/) to lint all JavaScript code and follow rules specified in [@cypress/eslint-plugin-dev](./npm/eslint-plugin-cypress) plugin. -When you edit files, you can quickly fix all changed files before you commit using - -```bash -$ yarn lint-changed --fix -``` - -When committing files, we run a Git pre-commit hook to lint the staged JS files. See the [`lint-staged` project](https://github.com/okonet/lint-staged). -If this command fails, you may need to run `yarn lint-changed --fix` and commit those changes. +This project uses a Git pre-commit hook to lint staged files before committing. See the [`lint-staged` project](https://github.com/okonet/lint-staged) for details. +`lint-staged` will try to auto-fix any lint errors with `eslint --fix`, so if it fails, you must manually fix the lint errors before committing. We **DO NOT** use Prettier to format code. You can find [.prettierignore](.prettierignore) file that ignores all files in this repository. To ensure this file is loaded, please always open _the root repository folder_ in your text editor, otherwise your code formatter might execute, reformatting lots of source files. diff --git a/package.json b/package.json index aa432f92396e..2c8cdd996728 100644 --- a/package.json +++ b/package.json @@ -42,7 +42,6 @@ "get-next-version": "node scripts/get-next-version.js", "postinstall": "node ./scripts/run-postInstall.js", "lint": "lerna run lint --no-bail --concurrency 2", - "lint-changed": "lint-changed", "prepare-release-artifacts": "node ./scripts/prepare-release-artifacts.js", "npm-release": "node scripts/npm-release.js", "prestart": "yarn ensure-deps",