Skip to content

Commit

Permalink
build: remove "formatfix" task
Browse files Browse the repository at this point in the history
Since our eslint rules now are informed by our prettier config
(performed by the "eslint-plugin-prettier" package), `prettier --write`
is unnecessary because it's implictly done by `eslint --fix`.

https://github.com/prettier/eslint-plugin-prettier
  • Loading branch information
justinmk3 committed Jul 12, 2024
1 parent 42465e2 commit 3636963
Show file tree
Hide file tree
Showing 4 changed files with 0 additions and 4 deletions.
1 change: 0 additions & 1 deletion package.json
Original file line number Diff line number Diff line change
Expand Up @@ -32,7 +32,6 @@
"package": "npm run package -w packages/toolkit -w packages/amazonq",
"newChange": "echo 'Must specify subproject/workspace with -w packages/<subproject>' && false",
"createRelease": "echo 'Must specify subproject/workspace with -w packages/<subproject>' && false",
"formatfix": "prettier --write plugins && npm run formatfix -w packages/ --if-present",
"lint": "npm run lint -w packages/ --if-present",
"lintfix": "eslint -c .eslintrc.js --fix --ext .ts packages plugins && npm run formatfix",
"clean": "npm run clean -w packages/ -w plugins/",
Expand Down
1 change: 0 additions & 1 deletion packages/amazonq/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -54,7 +54,6 @@
"compile": "npm run clean && npm run buildScripts && webpack --mode development",
"compileOnly": "tsc -p ./",
"package": "ts-node ../../scripts/package.ts",
"formatfix": "prettier --ignore-path ../../.prettierignore --write src scripts",
"lint": "true",
"watch": "npm run clean && npm run buildScripts && tsc -watch -p ./",
"testCompile": "npm run clean && npm run buildScripts && npm run compileOnly",
Expand Down
1 change: 0 additions & 1 deletion packages/core/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -4031,7 +4031,6 @@
"testWeb": "npm run compileDev && c8 ts-node ./scripts/test/testWeb.ts",
"testE2E": "npm run testCompile && c8 ts-node ./scripts/test/testE2E.ts dist/src/testE2E/index.js",
"testInteg": "npm run testCompile && c8 ts-node ./scripts/test/testInteg.ts",
"formatfix": "prettier --ignore-path ../../.prettierignore --write src scripts",
"lint": "ts-node ./scripts/lint/testLint.ts",
"generateClients": "ts-node ./scripts/build/generateServiceClient.ts ",
"generatePackage": "ts-node ./scripts/build/generateIcons.ts",
Expand Down
1 change: 0 additions & 1 deletion packages/toolkit/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -65,7 +65,6 @@
"webRun": "npx @vscode/test-web --open-devtools --browserOption=--disable-web-security --waitForDebugger=9222 --extensionDevelopmentPath=. .",
"package": "npm run copyPackageJson && ts-node ../../scripts/package.ts",
"install-plugin": "vsce package --ignoreFile '../.vscodeignore.packages' -o aws-toolkit-vscode-test.vsix && code --install-extension aws-toolkit-vscode-test.vsix",
"formatfix": "prettier --ignore-path ../../.prettierignore --write src scripts",
"lint": "true",
"createRelease": "ts-node ../../scripts/createRelease.ts",
"newChange": "ts-node ../../scripts/newChange.ts",
Expand Down

0 comments on commit 3636963

Please sign in to comment.