🛠 Tooling: Unnecessary --report-unused-disable-directives in templating #1089
Labels
area: tooling
Managing the repository's maintenance
good first issue
Good for newcomers, please hop on!
status: accepting prs
Please, send a pull request to resolve this!
type: cleanup
Tech debt or other code/repository cleanups
Bug Report Checklist
main
branch of the repository.Overview
Thanks to #1048, we see now that this repository has two bits of tooling that are different between its 🍎 actual files and 🍌 code templates (see Additional Info below for explanation of the emojis):
--report-unused-disable-directives
:create-typescript-app/package.json
Line 28 in d66379b
--report-unused-disable-directives
:create-typescript-app/src/steps/writing/creation/writePackageJson.ts
Line 81 in d66379b
There's no need for adding
--report-unused-disable-directives
in thelint
script, because it's now used in the ESLint config file: https://github.com/JoshuaKGoldberg/create-typescript-app/blob/d66379b285b542a14d87c4f53cd0f0ccbc5310e9/.eslintrc.cjs#L149C1-L149C1Let's remove any and all references to
--report-unused-disable-directives
from this repo. Accepting PRs!Additional Info
This repository is two things:
For the most part, the repository's end-to-end test for
--mode migration
makes sure that the 🍎 actual files and 🍌 code templates stay the same. As in, if one changes, the other needs to have the same change as well. But per #546, there are some files that were missed in that test. We're only now able to verify that they're the same as of #1048. That PR is where this discrepancy was discovered.The text was updated successfully, but these errors were encountered: