Skip to content

Apply typescript-eslint's recommended config to gts files#120

Merged
NullVoxPopuli merged 1 commit intoember-cli:mainfrom
bendemboski:fix-tseslint-recommended-gts
Dec 6, 2025
Merged

Apply typescript-eslint's recommended config to gts files#120
NullVoxPopuli merged 1 commit intoember-cli:mainfrom
bendemboski:fix-tseslint-recommended-gts

Conversation

@bendemboski
Copy link
Copy Markdown
Contributor

"Patch" typescript-eslint's recommended config to allow us to extend it to gts files, and also add test cases to verify that we're applying the ruleset to both ts and gts files.

Fixes #119

"Patch" `typescript-eslint`'s recommended config to allow us to extend it to `gts` files, and also add test cases to verify that we're applying the ruleset to both `ts` and `gts` files.

Fixes ember-cli#119
bendemboski added a commit to bendemboski/ember-app-blueprint that referenced this pull request Nov 24, 2025
"Patch" `typescript-eslint`'s recommended config to allow us to extend it to `gts` files, and also add test cases to verify that we're applying the ruleset to both `ts` and `gts` files.

This is the equivalent of ember-cli/ember-addon-blueprint#120 for this repo
Comment thread files/eslint.config.mjs
// https://github.com/ember-cli/ember-addon-blueprint/issues/119
{
...ts.configs.eslintRecommended,
files: undefined,
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks like this hack isn't needed?

eslint/eslint#20351

bendemboski added a commit to bendemboski/ember-app-blueprint that referenced this pull request Nov 26, 2025
"Patch" `typescript-eslint`'s recommended config to allow us to extend it to `gts` files, and also add test cases to verify that we're applying the ruleset to both `ts` and `gts` files.

This is the equivalent of ember-cli/ember-addon-blueprint#120 for this repo
@NullVoxPopuli NullVoxPopuli merged commit fe5d8e3 into ember-cli:main Dec 6, 2025
13 checks passed
@github-actions github-actions Bot mentioned this pull request Dec 6, 2025
@NullVoxPopuli NullVoxPopuli added the bug Something isn't working label Dec 6, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

typescript-eslint recommended rules don't apply to gts files

2 participants