Skip to content

ci: Pin react-router latest and add eslint disable for deprecated attribute#21624

Merged
nicohrubec merged 3 commits into
developfrom
test/pin-react-router-7-latest-canary
Jun 18, 2026
Merged

ci: Pin react-router latest and add eslint disable for deprecated attribute#21624
nicohrubec merged 3 commits into
developfrom
test/pin-react-router-7-latest-canary

Conversation

@nicohrubec

@nicohrubec nicohrubec commented Jun 18, 2026

Copy link
Copy Markdown
Member

CI is currently blocked with two unrelated failures:

  • Linting because of a deprecated attribute in our hapi instrumentation.
  • React router e2e tests because react router v8 was released.

Pinning react router and adding an eslint disable to unblock ci.

react-router 8 requires React 19 (imports useOptimistic); the SDK does not support v8 yet. Both the react-router-7-framework (latest) and -instrumentation E2E canaries float to v8 against React 18 and fail the build. Pin them to the newest v7 release until v8 is supported.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@nicohrubec nicohrubec force-pushed the test/pin-react-router-7-latest-canary branch from df14940 to 0dc5e03 Compare June 18, 2026 07:15
@nicohrubec nicohrubec changed the title test(react-router): Pin react-router latest canary to v7 test(react-router): Pin react-router latest canaries to v7 Jun 18, 2026
@nicohrubec nicohrubec marked this pull request as ready for review June 18, 2026 07:17
#21604 enabled typescript/no-deprecated and added disable comments across the codebase but missed this vendored hapi usage, breaking lint on develop.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@nicohrubec nicohrubec requested a review from a team as a code owner June 18, 2026 07:50
@nicohrubec nicohrubec requested review from JPeer264 and removed request for a team June 18, 2026 07:50
@nicohrubec nicohrubec changed the title test(react-router): Pin react-router latest canaries to v7 test(react-router): Pin react-router canaries to v7 + fix hapi no-deprecated lint Jun 18, 2026
@nicohrubec nicohrubec changed the title test(react-router): Pin react-router canaries to v7 + fix hapi no-deprecated lint ci: Pin react-router latest and add eslint disable for deprecated attribute Jun 18, 2026
@nicohrubec nicohrubec enabled auto-merge (squash) June 18, 2026 07:57
@nicohrubec nicohrubec merged commit 839391f into develop Jun 18, 2026
176 checks passed
@nicohrubec nicohrubec deleted the test/pin-react-router-7-latest-canary branch June 18, 2026 08: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.

5 participants