Uh oh!
There was an error while loading. Please reload this page.
Bump @typescript-eslint/parser from 8.66.0 to 8.67.0 - #2736
Conversation
There was a problem hiding this comment.
Copilot wasn't able to review any files in this pull request.
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
Dependabot (@dependabot) rebase |
Bumps [@typescript-eslint/parser](https://github.com/typescript-eslint/typescript-eslint/tree/HEAD/packages/parser) from 8.66.0 to 8.67.0. - [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.67.0/packages/parser) --- updated-dependencies: - dependency-name: "@typescript-eslint/parser" dependency-version: 8.67.0 dependency-type: direct:development update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com>
ebddc31 to
61ba20cCompareCopilot This PR was opened by Dependabot. Please take it the rest of the way before it's ready for human review:
|
Co-authored-by: jainakanksha-msft <181211853+jainakanksha-msft@users.noreply.github.com>
Addressed in feadc9c. This lockfile-only parser update (8.66.0 → 8.67.0) affects TypeScript lint parsing through |
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 2 out of 3 changed files in this pull request and generated no new comments.
Suppressed comments (1)
tests/eslintConfig.test.ts:13
- This test asserts only
errorCount === 0. If ESLint ends up ignoring the providedfilePath, ESLint can still returnerrorCount = 0(while emitting a warning like "File ignored...") and the test would pass without actually validating the parser/config. Consider also assertingwarningCount === 0(or thatresult.messagesis empty) so the smoke test fails when the file is ignored or any warnings are produced.
assert.strictEqual(result.errorCount, 0, JSON.stringify(result.messages));
Copilot, please handle this |
Co-authored-by: jainakanksha-msft <181211853+jainakanksha-msft@users.noreply.github.com>
... Added a |
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 2 out of 3 changed files in this pull request and generated no new comments.
Suppressed comments (1)
ChangeLog.md:9
- The changelog entry describes this as a devDependency bump from 8.66.0 to 8.67.0, but
package.jsonstill specifies@typescript-eslint/parseras^8.65.0. To avoid implying that the dependency spec was changed, consider wording this as an update to the lockfile-resolved version.
- Bumped `@typescript-eslint/parser` dev dependency from 8.66.0 to 8.67.0; added an ESLint TypeScript parsing smoke test to validate the updated parser configuration.
|
Co-authored-by: jainakanksha-msft <181211853+jainakanksha-msft@users.noreply.github.com>
... Updated the entry in 6586154 to describe the lockfile-resolved version change rather than a manifest dependency-spec bump. The focused parser smoke test passes. |
Uh oh!
There was an error while loading. Please reload this page.
Bumps @typescript-eslint/parser from 8.66.0 to 8.67.0.
Release notes
Sourced from @typescript-eslint/parser's releases.
Changelog
Sourced from @typescript-eslint/parser's changelog.
Commits
20a261fchore(release): publish 8.67.03b155bbchore: use typescript 7 for typechecking (#12601)