-
Notifications
You must be signed in to change notification settings - Fork 25
chore(deps-dev): bump typescript-eslint from 7.17.0 to 8.3.0 #310
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
chore(deps-dev): bump typescript-eslint from 7.17.0 to 8.3.0 #310
Conversation
303c50e to
b14e3e9
Compare
Bumps [typescript-eslint](https://github.com/typescript-eslint/typescript-eslint/tree/HEAD/packages/typescript-eslint) from 7.17.0 to 8.3.0. - [Release notes](https://github.com/typescript-eslint/typescript-eslint/releases) - [Changelog](https://github.com/typescript-eslint/typescript-eslint/blob/main/packages/typescript-eslint/CHANGELOG.md) - [Commits](https://github.com/typescript-eslint/typescript-eslint/commits/v8.3.0/packages/typescript-eslint) --- updated-dependencies: - dependency-name: typescript-eslint dependency-type: direct:development update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <support@github.com>
b14e3e9 to
c60e166
Compare
|
Upgrading to eslint 8.3.0 requires code changes, as the following new warning appears: This would need to be fixed in order to merge this. |
… configs between js and ts
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This file now contains static analysis rules only for JavaScript
| "//eslint": "performs static analysis over JS files and examples", | ||
| "eslint": "eslint --config eslint.config.mjs --fix test/*.js examples/**/*.js", | ||
| "//tseslint": "performs static analysis over TS files and examples", | ||
| "tseslint": "eslint --config tseslint.config.mjs --fix lib/*.ts", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The eslint command is now split in eslint and tseslint to use different config files per language
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The new tseslint.config.mjs is basically the old eslint.config.mjs
| # Run static analysis on JS files | ||
| - run: npm run eslint | ||
| # Run static analysis on TS files | ||
| - run: npm run tseslint |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
sumitramanga
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM! Thank you for intervening to get this through 💪🏽

Bumps typescript-eslint from 7.17.0 to 8.3.0.
Release notes
Sourced from typescript-eslint's releases.
... (truncated)
Changelog
Sourced from typescript-eslint's changelog.
... (truncated)
Commits
ef2eab1chore(release): publish 8.3.0d4f6943chore: enable eslint-plugin-perfectionist on typescript-eslint package (#9851)57e4120feat(eslint-plugin): [no-deprecation] add rule (#9783)7440e14chore(release): publish 8.2.03d78392chore(release): publish 8.1.090576fafeat(typescript-eslint): add JSDoc comments on generated configs (#9672)fb65883chore: setreportUnusedDisableDirectiveslevel toerror(#9743)e05c2e5chore(release): publish 8.0.11db993achore(release): publish 8.0.01b0468eMerge branch 'main'Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting
@dependabot rebase.Dependabot commands and options
You can trigger Dependabot actions by commenting on this PR:
@dependabot rebasewill rebase this PR@dependabot recreatewill recreate this PR, overwriting any edits that have been made to it@dependabot mergewill merge this PR after your CI passes on it@dependabot squash and mergewill squash and merge this PR after your CI passes on it@dependabot cancel mergewill cancel a previously requested merge and block automerging@dependabot reopenwill reopen this PR if it is closed@dependabot closewill close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually@dependabot show <dependency name> ignore conditionswill show all of the ignore conditions of the specified dependency@dependabot ignore this major versionwill close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)@dependabot ignore this minor versionwill close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)@dependabot ignore this dependencywill close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)