build: switch Makefiles and pre-commit to flat config (gh54 part VII)#10973
Draft
Planeshifter wants to merge 1 commit intophilipp/gh54-eslint-v9from
Draft
build: switch Makefiles and pre-commit to flat config (gh54 part VII)#10973Planeshifter wants to merge 1 commit intophilipp/gh54-eslint-v9from
Planeshifter wants to merge 1 commit intophilipp/gh54-eslint-v9from
Conversation
Planeshifter
changed the title
Planeshifter
force-pushed
the
philipp/gh54-eslint-v9
branch
from
85dcf2f to
fe1dbb3
Compare
Update build tooling to use ESLint flat config (eslint.config.cjs) instead of legacy .eslintrc.* files: - Remove `ESLINT_USE_FLAT_CONFIG=false` from JS eslint.mk - Remove `--ignore-path`, `--config`, and per-target config variables from both JS and TS eslint.mk (flat config autodiscovery handles all of this via eslint.config.cjs) - Remove per-directory tsconfig.json copy mechanism from TS eslint.mk (the flat config's TS block points to the root tsconfig.json) - Remove legacy config path variables from pre-commit hook ESLint now autodiscovers eslint.config.cjs at the repo root and applies the correct rules based on file patterns (benchmarks, tests, examples, TypeScript) defined in the flat config. Ref: stdlib-js/metr-issue-tracker#54
Planeshifter
force-pushed
the
philipp/gh54-switch-to-flat-config
branch
from
c3dab8a to
c6bab92
Compare
7 tasks
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Progresses stdlib-js/metr-issue-tracker#54.
Description
This pull request:
.eslintrc.*config to flat config autodiscoveryESLINT_USE_FLAT_CONFIG=false,--ignore-path,--config, and per-target config variables from JS and TS eslint.mk.tsconfig.jsoncopy mechanism from TS eslint.mk (flat config's TS block handles this).eslint.config.cjsand applies rules based on file patterns.Related Issues
This pull request has the following related issues:
Questions
No.
Other
Stacked on #10972.
Checklist
AI Assistance
If you answered "yes" above, how did you use AI assistance?
Disclosure
This PR was written primarily by Claude Code.
@stdlib-js/reviewers