Skip to content

Conversation

trentm
Copy link
Member

@trentm trentm commented Sep 4, 2025

This removes Next.js instrumentation, which was only ever in tech preview
and only supported up to Next.js 13.3.0, which is now obsolete. The
built-in OpenTelemetry instrumentation in Next.js is a better option
users.

This removes Next.js instrumentation, which was only ever in tech preview
and only supported up to Next.js 13.3.0, which is now obsolete. The
built-in OpenTelemetry instrumentation in Next.js is a better option
users.
@trentm trentm self-assigned this Sep 4, 2025
Copy link

github-actions bot commented Sep 4, 2025

🤖 GitHub comments

Expand to view the GitHub comments

Just comment with:

  • run docs-build : Re-trigger the docs validation. (use unformatted text in the comment!)

@trentm
Copy link
Member Author

trentm commented Sep 4, 2025

Aside: Most of our dependabot "security" issues are from the out-dated examples/nextjs and Next.js instr tests. While this is breaking, it only ever supported Next.js versions up to 13.3.0, which stopped getting support in 2022. Also, our instrumentation support here was only ever a tech preview, so I believe we can drop it in a minor.

Copy link

github-actions bot commented Sep 4, 2025

@trentm trentm marked this pull request as ready for review September 4, 2025 19:25
@trentm trentm requested a review from a team as a code owner September 4, 2025 19:25
david-luna
david-luna previously approved these changes Sep 10, 2025
@david-luna
Copy link
Member

not: dependabot.yml has an ignore rule for next in

- dependency-name: "next" # support is >=12.0.0 <13.3.0

@trentm
Copy link
Member Author

trentm commented Sep 10, 2025

dependabot.yml has an ignore rule for next in

Thanks. I need to remove that.

@trentm trentm requested a review from david-luna September 10, 2025 17:09
david-luna
david-luna previously approved these changes Sep 10, 2025
david-luna
david-luna previously approved these changes Sep 11, 2025
david-luna
david-luna previously approved these changes Sep 19, 2025
david-luna
david-luna previously approved these changes Sep 19, 2025
@trentm trentm merged commit 9ccaa33 into main Sep 19, 2025
18 checks passed
@trentm trentm deleted the trentm-rm-nextjs-instr branch September 19, 2025 17:09
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