| FazBrowse GitHub Viewer | Trending | | Home |
| Tools: [Download Repo ZIP] [Original HTTPS Page] |
Bumps the npm group with 2 updates in the / directory: [@typescript-eslint/eslint-plugin](https://github.com/typescript-eslint/typescript-eslint/tree/HEAD/packages/eslint-plugin) and [@typescript-eslint/parser](https://github.com/typescript-eslint/typescript-eslint/tree/HEAD/packages/parser). Updates `@typescript-eslint/eslint-plugin` from 7.17.0 to 8.0.1 - [Release notes](https://github.com/typescript-eslint/typescript-eslint/releases) - [Changelog](https://github.com/typescript-eslint/typescript-eslint/blob/main/packages/eslint-plugin/CHANGELOG.md) - [Commits](https://github.com/typescript-eslint/typescript-eslint/commits/v8.0.1/packages/eslint-plugin) Updates `@typescript-eslint/parser` from 7.17.0 to 8.0.1 - [Release notes](https://github.com/typescript-eslint/typescript-eslint/releases) - [Changelog](https://github.com/typescript-eslint/typescript-eslint/blob/main/packages/parser/CHANGELOG.md) - [Commits](https://github.com/typescript-eslint/typescript-eslint/commits/v8.0.1/packages/parser) --- updated-dependencies: - dependency-name: "@typescript-eslint/eslint-plugin" dependency-type: direct:development update-type: version-update:semver-major dependency-group: npm - dependency-name: "@typescript-eslint/parser" dependency-type: direct:development update-type: version-update:semver-major dependency-group: npm ... Signed-off-by: dependabot[bot] <support@github.com>
|
Pushed a commit to update the checked-in dependencies. Please mark the PR as ready for review to trigger PR checks. |
Sorry, something went wrong.
|
cc @NlightNFotis since you're on 🛡️ duty this week. Would you mind reviewing my changes? |
Sorry, something went wrong.
There was a problem hiding this comment.
LGTM, just one question.
Sorry, something went wrong.
| const safeWhich = __importStar(require("@chrisgavin/safe-which")); | ||
| const util_1 = require("./util"); | ||
| // eslint-disable-next-line import/no-commonjs | ||
| // eslint-disable-next-line import/no-commonjs, @typescript-eslint/no-require-imports |
There was a problem hiding this comment.
Why are we getting the typescript-eslint alert if we already have the eslint import/no-commonjs one? Is it a matter of it being copied in the js file by the TypeScript compiler?
Sorry, something went wrong.
There was a problem hiding this comment.
I think there's some duplication within our set of enabled ESLint rules that is causing this problem to be picked up by two rules. We should take a look at the set of enabled rules as a low priority followup.
Sorry, something went wrong.
| Back | FazBrowse Home | New Git URL |
Bumps the npm group with 2 updates in the / directory: @typescript-eslint/eslint-plugin and @typescript-eslint/parser.
Updates @typescript-eslint/eslint-plugin from 7.17.0 to 8.0.1
Release notesSourced from @typescript-eslint/eslint-plugin's releases.
... (truncated)
ChangelogSourced from @typescript-eslint/eslint-plugin's changelog.
... (truncated)
CommitsUpdates @typescript-eslint/parser from 7.17.0 to 8.0.1
Release notesSourced from @typescript-eslint/parser's releases.
... (truncated)
ChangelogSourced from @typescript-eslint/parser's changelog.
CommitsDependabot 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: