-
Notifications
You must be signed in to change notification settings - Fork 0
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
Update dependency @sentry/core to v8.51.0 #775
Conversation
Important Review skippedBot user detected. To trigger a single review, invoke the You can disable this status message by setting the Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
Documentation and Community
|
fd6f317
to
40e2ff7
Compare
@coderabbitai why isn't the pnpm-lock updated too? Am I missing some configuration? |
Tip For best results, initiate chat on the files or code changes. @bradleyayers The pnpm-lock.yaml file isn't being updated because this project uses Yarn as its package manager (specifically Yarn v4.6.0), not pnpm. This is defined in your package.json: "packageManager": "[email protected]" Renovate should be updating the yarn.lock file instead. Would you like me to check if the yarn.lock file is being updated correctly? 🔗 Analysis chain🏁 Scripts executedThe following scripts were executed for the analysis: Script:
Length of output: 6056 |
@coderabbitai I have both yarn and pnpm in the repository. |
Tip For best results, initiate chat on the files or code changes. @bradleyayers The issue appears to be that Renovate isn't detecting or handling the nested pnpm setup properly. While Renovate should automatically handle multiple package managers, it might need explicit configuration in this case due to the nested structure. You can add the following to your {
"enabledManagers": ["npm", "pnpm"],
"lockFileMaintenance": {
"enabled": true,
"extends": ["group:all"]
}
} This will explicitly enable both npm (yarn) and pnpm managers and ensure lock file maintenance is performed for both. Would you like me to create an issue to track this configuration update? 🔗 Analysis chain@bradleyayers Ah, I see the situation now! You're correct - both package managers are used in the repository:
Since @sentry/node is used in both:
Both lock files should indeed be updated. Let me check if Renovate is properly configured to handle this. 🏁 Scripts executedThe following scripts were executed for the analysis: Script:
Length of output: 6428 Script:
Length of output: 1493 Script:
Length of output: 1399 Script:
Length of output: 278 ✏️ Learnings added
|
@coderabbitai yes please make this change in a new PR |
Tip For best results, initiate chat on the files or code changes. @bradleyayers I've created issue #778 to track the Renovate configuration update that will handle both Yarn and pnpm lock files. 🔗 Analysis chain🏁 Scripts executedThe following scripts were executed for the analysis: Script:
Length of output: 870 🧠 Learnings used
|
74c39cd
to
11cc79b
Compare
baf650d
to
2aaf298
Compare
6ff574b
to
48e3c74
Compare
58e88c1
to
9c22601
Compare
45b6e62
to
24ee840
Compare
24ee840
to
412a8d2
Compare
This PR contains the following updates:
8.48.0
->8.51.0
Release Notes
getsentry/sentry-javascript (@sentry/core)
v8.51.0
Compare Source
Important Changes
feat(v8/node): Add
prismaInstrumentation
option to Prisma integration as escape hatch for all Prisma versions (#15128)This release adds a compatibility API to add support for Prisma version 6.
To capture performance data for Prisma version 6:
Install the
@prisma/instrumentation
package on version 6.Pass a
new PrismaInstrumentation()
instance as exported from@prisma/instrumentation
to theprismaInstrumentation
option:The passed instrumentation instance will override the default instrumentation instance the integration would use, while the
prismaIntegration
will still ensure data compatibility for the various Prisma versions.Remove the
previewFeatures = ["tracing"]
option from the client generator block of your Prisma schema.Other Changes
multiplexedtransport.js
CDN bundle (#15046)fetchProxyScriptNonce
option (#15011)fatal
events should set session as crashed (#15073)Work in this release was contributed by @tjhiggins, and @nwalters512. Thank you for your contributions!
Bundle size 📦
v8.50.0
Compare Source
createMemoryRouter
(#14985)Bundle size 📦
v8.49.0
Compare Source
handled
prop to ErrorBoundary (#14978)require
,__filename
and__dirname
on global object (#14952)Work in this release was contributed by @HHK1 and @mstrokin. Thank you for your contribution!
Bundle size 📦
Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Enabled.
♻ Rebasing: Whenever PR is behind base branch, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.
This PR was generated by Mend Renovate. View the repository job log.