Skip to content

Adopt ESLint 10 recommended rules and ES2024 - #1749

Merged
jeffwidman merged 1 commit into
mainfrom
eslint-v10-recommended-es2024
Aug 10, 2026
Merged

jeffwidman merged 1 commit into
mainfrom
eslint-v10-recommended-es2024

Conversation

@jeffwidman

Copy link
Copy Markdown
Member

Summary

  • enable the @eslint/js v10 recommended rules
  • align TypeScript and ESLint language levels with the Node 24 runtime
  • replace a direct hasOwnProperty call reported by the expanded rules
  • rebuild the checked-in action bundle

Validation

  • npm run lint-check
  • npx tsc --noEmit
  • npm run format-check
  • npm test
  • npm run package
  • git diff --check

@jeffwidman
jeffwidman requested a review from a team as a code owner August 10, 2026 19:00
Copilot AI balanced review requested due to automatic review settings August 10, 2026 19:00

Copilot AI left a comment

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.

Pull request overview

Adopts ESLint 10 recommended rules and aligns compilation with Node 24’s ES2024 runtime.

Changes:

  • Enables recommended ESLint rules and ES2024 parsing.
  • Updates the TypeScript target to ES2024.
  • Replaces unsafe hasOwnProperty usage and rebuilds the action bundle.
Show a summary per file
File Description
tsconfig.json Targets ES2024.
src/updater.ts Uses Object.hasOwn safely.
eslint.config.mjs Enables recommended rules and ES2024 syntax.
dist/main/index.js Updates the packaged action bundle.

Review details

Tip

Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

  • Files reviewed: 2/5 changed files
  • Comments generated: 0
  • Review effort level: Balanced

@jeffwidman
jeffwidman merged commit 0764419 into main Aug 10, 2026
11 of 13 checks passed
@jeffwidman
jeffwidman deleted the eslint-v10-recommended-es2024 branch August 10, 2026 19:29
@pavera pavera mentioned this pull request Aug 27, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants