Conversation
WalkthroughThis pull request updates a dependency in apps/api/v2/package.json, changing @calcom/platform-libraries from version 0.0.349 to 0.0.350 in the dependencies section. No other files or configurations are modified. Possibly related PRs
Tip 👮 Agentic pre-merge checks are now available in preview!Pro plan users can now enable pre-merge checks in their settings to enforce checklists before merging PRs.
Please see the documentation for more information. Example: reviews:
pre_merge_checks:
custom_checks:
- name: "Undocumented Breaking Changes"
mode: "warning"
instructions: |
Pass/fail criteria: All breaking changes to public APIs, CLI flags, environment variables, configuration keys, database schemas, or HTTP/GraphQL endpoints must be documented in the "Breaking Change" section of the PR description and in CHANGELOG.md. Exclude purely internal or private changes (e.g., code not exported from package entry points or explicitly marked as internal).Please share your feedback with us on this Discord post. Pre-merge checks (2 passed, 1 inconclusive)❌ Failed checks (1 inconclusive)
✅ Passed checks (2 passed)
✨ Finishing Touches
🧪 Generate unit tests
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 |
There was a problem hiding this comment.
Actionable comments posted: 0
🧹 Nitpick comments (1)
apps/api/v2/package.json (1)
12-18: Double-check workspace vs npm alias resolution for @calcom/platform-libraries
Dependencies in apps/api/v2/package.json use thenpm:protocol, while your dev scripts invoke the local workspace—this can lead to duplicate installs or version drift. Verify how Yarn actually resolves this by inspecting your declaration and lockfile:grep -R '"@calcom/platform-libraries"' -n apps/api/v2/package.json yarn workspaces list --json | jq '.[] | select(.name=="@calcom/platform-libraries")' yarn why @calcom/platform-libraries grep -R '@calcom/platform-libraries@' -n yarn.lockIf you intend to track the published package at runtime, keep
npm:and add a comment explaining this choice. If you want local linking during development, switch the dependency toworkspace:*for consistency.
📜 Review details
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Pro
💡 Knowledge Base configuration:
- Linear integration is disabled by default for public repositories
You can enable these sources in your CodeRabbit configuration.
⛔ Files ignored due to path filters (1)
yarn.lockis excluded by!**/yarn.lock,!**/*.lock
📒 Files selected for processing (1)
apps/api/v2/package.json(1 hunks)
⏰ Context from checks skipped due to timeout of 180000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (2)
- GitHub Check: Codacy Static Code Analysis
- GitHub Check: Detect changes
🔇 Additional comments (1)
apps/api/v2/package.json (1)
41-41: Upgrade verified: @calcom/platform-libraries@0.0.350 present, no stale 0.0.349 refs, and lockfiles clean.
E2E results are ready! |
Published ✅