chore(devtools): bump packages and move theme provider#176
Conversation
🦋 Changeset detectedLatest commit: 30f8d0e The changes in this PR will be included in the next version bump. This PR includes changesets to release 4 packages
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (6)
📝 WalkthroughWalkthroughThis PR extracts the devtools theme provider to respective devtools packages to prevent theme mismatch. Dependencies across pacer, preact, react, and solid devtools packages are updated. The PacerDevtools component is refactored to accept a theme prop and wrap its content with ThemeContextProvider. Changes
Sequence DiagramsequenceDiagram
participant Parent as Parent Component
participant Devtools as PacerDevtools
participant Provider as ThemeContextProvider
participant Shell as Shell & Context
Parent->>Devtools: Pass theme prop
Devtools->>Provider: Wrap with ThemeContextProvider
Note over Provider: Provide theme to context
Provider->>Shell: Render with theme context
Shell->>Shell: Access theme from context
Estimated code review effort🎯 2 (Simple) | ⏱️ ~12 minutes Poem
🚥 Pre-merge checks | ✅ 1 | ❌ 2❌ Failed checks (2 warnings)
✅ Passed checks (1 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches
🧪 Generate unit tests (beta)
📝 Coding Plan
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
View your CI Pipeline Execution ↗ for commit 30f8d0e
☁️ Nx Cloud last updated this comment at |
3 participants
Summary by CodeRabbit
New Features
Chores