Skip to content

feat(nodejs): improve eslint setup#2194

Merged
wpessers merged 7 commits intoopen-telemetry:mainfrom
wpessers:chore/upgrade-eslint
Mar 22, 2026
Merged

feat(nodejs): improve eslint setup#2194
wpessers merged 7 commits intoopen-telemetry:mainfrom
wpessers:chore/upgrade-eslint

Conversation

@wpessers
Copy link
Member

fixes #2158

Changes in this PR:

  • Upgraded eslint to major version 9
  • Upgraded gts to major version 7
  • Removed @typescript-eslint/eslint-plugin and @typescript-eslint/parser, they are now bundled by gts
  • Rewrote eslint config to use flat config, consolidating all subproject configs and .eslintignore files
  • Applied formatting to .js, .cjs, .mjs files as well
  • (re)formatted wherever necessary. Lots of changes to the dynamic imports' formatting in wrapper.ts due to an underlying upgrade of prettier: https://prettier.io/blog/2025/11/27/3.7.0#change-17882

@wpessers wpessers requested a review from a team as a code owner March 22, 2026 16:30
@wpessers
Copy link
Member Author

@pragmaticivan maybe you can take a look here, seeing your js expertise 🙂

@wpessers
Copy link
Member Author

Or anyone else in @open-telemetry/lambda-extension-approvers who's eager to take a look 🙂

Copy link
Member

@maxday maxday left a comment

Choose a reason for hiding this comment

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

thannnks for doing this!

@wpessers wpessers merged commit c90b1dd into open-telemetry:main Mar 22, 2026
13 of 14 checks passed
@wpessers wpessers added enhancement New feature or request dependencies Pull requests that update a dependency file javascript Pull requests that update Javascript code labels Mar 22, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file enhancement New feature or request javascript Pull requests that update Javascript code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Upgrade eslint

2 participants