docs: update theming example to next 15 #6209
Conversation
|
size-limit report 📦
|
There was a problem hiding this comment.
Pull Request Overview
This PR updates the theming example from Next.js v14 to v15, ensuring compatibility with the latest Next.js features.
- Added a
targetof ES2017 to the TypeScript config. - Converted
RootLayoutto an async component to await the newcookies()API. - Bumped
next,react,react-dom, and ESLint plugin versions; loosened the@primer/reactversion specifier.
Reviewed Changes
Copilot reviewed 5 out of 5 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| examples/theming/tsconfig.json | Added "target": "ES2017" under compilerOptions |
| examples/theming/src/app/layout.tsx | Marked RootLayout async and awaited cookies() |
| examples/theming/package.json | Updated dependencies to Next.js v15 and caret ranges |
| @@ -11,16 +11,16 @@ | |||
| }, | |||
| "dependencies": { | |||
| "@primer/octicons-react": "^19.14.0", | |||
| "@primer/react": "37.26.0", | |||
| "@primer/react": "^37.26.0", | |||
There was a problem hiding this comment.
[nitpick] Consider whether the example should pin @primer/react to an exact version or keep the caret range to maintain consistency with other example workspaces.
| "@primer/react": "^37.26.0", | |
| "@primer/react": "37.26.0", |
Copilot uses AI. Check for mistakes.
Update our theming example from Next v14 to v15.
Changelog
New
Changed
examples/themingfrom Next v14 to Next v15Removed
Rollout strategy
This change is only to an internal example and does not affect our Public API.