Skip to content

Conversation

@antonis
Copy link
Contributor

@antonis antonis commented Jan 30, 2026

📢 Type of change

  • Bugfix
  • New feature
  • Enhancement
  • Refactoring

Based on #5594

📜 Description

ESLint 9 requires typescript-eslint v8+, but the dependabot upgrade only bumped ESLint itself. This adds the necessary typescript-eslint upgrades and yarn resolutions to resolve the incompatibility with @sentry-internal/eslint-config-sdk which depends on v5. Also adds ESLINT_USE_FLAT_CONFIG=false to support legacy .eslintrc.js format.

💡 Motivation and Context

💚 How did you test it?

📝 Checklist

  • I added tests to verify changes
  • No new PII added or SDK only sends newly added PII if sendDefaultPII is enabled
  • I updated the docs if needed.
  • I updated the wizard if needed.
  • All tests passing
  • No breaking changes

🔮 Next steps

#skip-changelog

dependabot bot and others added 2 commits January 29, 2026 15:17
Bumps [eslint](https://github.com/eslint/eslint) from 8.57.1 to 9.26.0.
- [Release notes](https://github.com/eslint/eslint/releases)
- [Changelog](https://github.com/eslint/eslint/blob/v9.26.0/CHANGELOG.md)
- [Commits](eslint/eslint@v8.57.1...v9.26.0)

---
updated-dependencies:
- dependency-name: eslint
  dependency-version: 9.26.0
  dependency-type: direct:development
...

Signed-off-by: dependabot[bot] <support@github.com>
…lity

ESLint 9 requires typescript-eslint v8+, but the dependabot upgrade only bumped ESLint itself. This adds the necessary typescript-eslint upgrades and yarn resolutions to resolve the incompatibility with @sentry-internal/eslint-config-sdk which depends on v5. Also adds ESLINT_USE_FLAT_CONFIG=false to support legacy .eslintrc.js format.

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
@github-actions
Copy link
Contributor

github-actions bot commented Jan 30, 2026

Semver Impact of This PR

None (no version bump detected)

📋 Changelog Preview

This is how your changes will appear in the changelog.
Entries from this PR are highlighted with a left border (blockquote style).


This PR will not appear in the changelog.


🤖 This preview updates automatically when you update the PR.

@antonis antonis added the ready-to-merge Triggers the full CI test suite label Jan 30, 2026
@antonis
Copy link
Contributor Author

antonis commented Jan 30, 2026

@sentry review

@github-actions
Copy link
Contributor

Android (new) Performance metrics 🚀

  Plain With Sentry Diff
Startup time 475.87 ms 511.09 ms 35.22 ms
Size 43.94 MiB 49.26 MiB 5.32 MiB

@github-actions
Copy link
Contributor

Android (legacy) Performance metrics 🚀

  Plain With Sentry Diff
Startup time 464.65 ms 499.64 ms 34.99 ms
Size 43.75 MiB 48.40 MiB 4.65 MiB

@github-actions
Copy link
Contributor

iOS (new) Performance metrics 🚀

  Plain With Sentry Diff
Startup time 1211.87 ms 1213.47 ms 1.60 ms
Size 3.38 MiB 4.60 MiB 1.22 MiB

@antonis antonis removed the ready-to-merge Triggers the full CI test suite label Jan 30, 2026
@github-actions
Copy link
Contributor

iOS (legacy) Performance metrics 🚀

  Plain With Sentry Diff
Startup time 1214.65 ms 1212.02 ms -2.63 ms
Size 3.38 MiB 4.60 MiB 1.22 MiB

@github-actions
Copy link
Contributor

Fails
🚫 Pull request is not ready for merge, please add the "ready-to-merge" label to the pull request

Generated by 🚫 dangerJS against 56aa1de

@dependabot dependabot bot force-pushed the dependabot/npm_and_yarn/eslint-9.26.0 branch from 02d0b55 to 45b285a Compare January 30, 2026 09:12
Copy link
Contributor Author

@antonis antonis left a comment

Choose a reason for hiding this comment

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

Closing this for now since it seems to require many source changes that would escape the scope of #5594

@antonis antonis closed this Jan 30, 2026
@antonis antonis deleted the antonis/eslint-9.26.0 branch January 30, 2026 09:16
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.

2 participants