-
Notifications
You must be signed in to change notification settings - Fork 202
chore: bumping expr dependency #2419
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
WalkthroughUpdated the indirect dependency Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes Possibly related PRs
Pre-merge checks and finishing touches✅ Passed checks (3 passed)
✨ Finishing touches
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 |
Router-nonroot image scan passed✅ No security vulnerabilities found in image: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 1
📜 Review details
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro
⛔ Files ignored due to path filters (2)
router-tests/go.sumis excluded by!**/*.sumrouter/go.sumis excluded by!**/*.sum
📒 Files selected for processing (3)
demo/go.mod(1 hunks)router-tests/go.mod(1 hunks)router/go.mod(1 hunks)
🧰 Additional context used
🧠 Learnings (3)
📚 Learning: 2025-09-24T12:54:00.765Z
Learnt from: endigma
Repo: wundergraph/cosmo PR: 2222
File: router-tests/websocket_test.go:2238-2302
Timestamp: 2025-09-24T12:54:00.765Z
Learning: The wundergraph/cosmo project uses Go 1.25 (Go 1.25 minimum), so fmt.Appendf and other newer Go standard library functions are available and can be used without compatibility concerns.
Applied to files:
demo/go.mod
📚 Learning: 2025-09-24T12:54:00.765Z
Learnt from: endigma
Repo: wundergraph/cosmo PR: 2222
File: router-tests/websocket_test.go:2238-2302
Timestamp: 2025-09-24T12:54:00.765Z
Learning: The wundergraph/cosmo project uses Go 1.25 (Go 1.23+ minimum), so fmt.Appendf and other newer Go standard library functions are available and can be used without compatibility concerns.
Applied to files:
demo/go.mod
📚 Learning: 2025-08-20T22:13:25.222Z
Learnt from: StarpTech
Repo: wundergraph/cosmo PR: 2157
File: router-tests/go.mod:16-16
Timestamp: 2025-08-20T22:13:25.222Z
Learning: github.com/mark3labs/mcp-go v0.38.0 has regressions and should not be used in the wundergraph/cosmo project. v0.36.0 is the stable version that should be used across router-tests and other modules.
Applied to files:
router/go.modrouter-tests/go.mod
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (12)
- GitHub Check: build-router
- GitHub Check: build_test
- GitHub Check: image_scan (nonroot)
- GitHub Check: integration_test (./events)
- GitHub Check: build_push_image
- GitHub Check: integration_test (./. ./fuzzquery ./lifecycle ./modules)
- GitHub Check: integration_test (./telemetry)
- GitHub Check: image_scan
- GitHub Check: build_push_image (nonroot)
- GitHub Check: build_test
- GitHub Check: Analyze (javascript-typescript)
- GitHub Check: Analyze (go)
🔇 Additional comments (2)
demo/go.mod (1)
57-57: Indirect dependency aligned with direct dependencies.The indirect dependency update is aligned with the patch bump across all modules.
router-tests/go.mod (1)
73-73: Consistent version update across all modules.The test module's indirect dependency is updated consistently with the main changes.
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #2419 +/- ##
===========================================
- Coverage 90.55% 43.69% -46.86%
===========================================
Files 16 229 +213
Lines 3271 23812 +20541
Branches 871 0 -871
===========================================
+ Hits 2962 10405 +7443
- Misses 309 12320 +12011
- Partials 0 1087 +1087 🚀 New features to boost your workflow:
|
Noroth
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
…ueries (wundergraph#2411) chore(release): Publish [skip ci] - wgc@0.102.5 - @wundergraph/composition@0.49.0 - controlplane@0.181.0 - graphqlmetrics@0.37.0 - @wundergraph/protographic@0.14.0 - router@0.273.0 - @wundergraph/cosmo-shared@0.42.24 - studio@0.147.0 feat: upgrade NextJS (wundergraph#2410) chore(release): Publish [skip ci] - studio@0.148.0 fix: resolve js-yaml vulnerabilities (wundergraph#2415) feat: validate session cookie (wundergraph#2406) Co-authored-by: Milinda Dias <83293842+SkArchon@users.noreply.github.com> chore(release): Publish [skip ci] - wgc@0.102.6 - controlplane@0.182.0 - keycloak@0.11.1 fix: codecov carryforward paths are not detected (wundergraph#2412) chore: bumping expr dependency (wundergraph#2419) fix: make error accessible to custom modules (wundergraph#2420) chore(release): Publish [skip ci] - router@0.273.1 fix: use the correct cgroup policy to work on most of the systems (wundergraph#2421) chore(release): Publish [skip ci] - router@0.273.2 Add support for per message deflate to websocket connectinos Merge branch 'main' into per-message-deflate revert formatting
…ueries (wundergraph#2411) chore(release): Publish [skip ci] - wgc@0.102.5 - @wundergraph/composition@0.49.0 - controlplane@0.181.0 - graphqlmetrics@0.37.0 - @wundergraph/protographic@0.14.0 - router@0.273.0 - @wundergraph/cosmo-shared@0.42.24 - studio@0.147.0 feat: upgrade NextJS (wundergraph#2410) chore(release): Publish [skip ci] - studio@0.148.0 fix: resolve js-yaml vulnerabilities (wundergraph#2415) feat: validate session cookie (wundergraph#2406) Co-authored-by: Milinda Dias <83293842+SkArchon@users.noreply.github.com> chore(release): Publish [skip ci] - wgc@0.102.6 - controlplane@0.182.0 - keycloak@0.11.1 fix: codecov carryforward paths are not detected (wundergraph#2412) chore: bumping expr dependency (wundergraph#2419) fix: make error accessible to custom modules (wundergraph#2420) chore(release): Publish [skip ci] - router@0.273.1 fix: use the correct cgroup policy to work on most of the systems (wundergraph#2421) chore(release): Publish [skip ci] - router@0.273.2 Add support for per message deflate to websocket connectinos Merge branch 'main' into per-message-deflate revert formatting
Summary by CodeRabbit
✏️ Tip: You can customize this high-level summary in your review settings.