Bump Sentry from 6.6.0 to 6.8.0 - #365
Conversation
--- updated-dependencies: - dependency-name: Sentry dependency-version: 6.8.0 dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com>
dependabot
Bot
added
automated
LabelsThe following labels could not be found: Please fix the above issues or remove invalid values from |
dependabot
Bot
added
dependencies
|
Tick the box to add this pull request to the merge queue (same as
|
test200 tests 200 ✅ 0s ⏱️ Results for commit df6a431. |
|
Review the following changes in direct dependencies. Learn more about Socket for GitHub.
|
![kilo-code-bot[bot]](https://avatars.githubusercontent.com/in/2193792?s=60&v=4){kind=small}
| @@ -368,16 +368,16 @@ | |||
| }, | |||
| "Serilog.Sinks.File": { | |||
| "type": "Transitive", | |||
There was a problem hiding this comment.
WARNING: Major version bump for transitive dependency Serilog.Sinks.File (6.0.0 → 7.0.0). DINOForge.Bridge.Client already declares Serilog.Sinks.File 7.*, so the lock file was stale before this update. Verify compatibility with Serilog.Sinks.File 7.x APIs before merging.
| "Serilog": "[4.*, )", | ||
| "Serilog.Sinks.Console": "[6.*, )", | ||
| "Serilog.Sinks.File": "[6.*, )" | ||
| "Serilog.Sinks.File": "[7.*, )" |
There was a problem hiding this comment.
SUGGESTION: Transitive dependency constraint for Serilog.Sinks.File in dinoforge.bridge.protocol changed from [6.*, ) to [7.*, ). This aligns with Bridge.Client's existing Serilog.Sinks.File 7.* reference, but the prior lock file was inconsistent with project declarations — suggests the lock file may need a proactive refresh.
Code Review SummaryStatus: 2 Issues Found | Recommendation: Merge with notes Overview
Issue Details (click to expand)WARNING
SUGGESTION
Files Reviewed (2 files)
Fix these issues in Kilo Cloud Reviewed by ling-3.0-flash:free · Input: 174.3K · Output: 29.3K · Cached: 486K |
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: df6a431262
ℹ️ About Codex in GitHub
Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
| "requested": "[8.*, )", | ||
| "resolved": "8.0.27", | ||
| "contentHash": "qlS+YrntAGokJdNNU9iEbf1cjhUw82xspqWq1J4haGrUk800HqHid5q9/dfkLGxNGYubT/MysPoIbo364et20Q==", | ||
| "resolved": "8.0.29", |
There was a problem hiding this comment.
Checked the CLI restore in .github/workflows/build-gate.yml lines 108-111: it uses RestoreLockedMode=true, so this lockfile determines the versions validated by CI. Besides Sentry, this change advances Microsoft.Data.Sqlite here and also System.Drawing.Common, Spectre.Console, System.CommandLine, Serilog, and Serilog.Sinks.File later in the file; those independent upgrades are therefore bundled into a nominally Sentry-only change without separate validation or attribution. Preserve their previous resolutions or split them into separately tested dependency updates.
AGENTS.md reference: AGENTS.md:L114-L120
Useful? React with 👍 / 👎.
|
|
||
| <!-- Error tracking --> | ||
| <PackageReference Include="Sentry" Version="6.6.0" /> | ||
| <PackageReference Include="Sentry" Version="6.8.0" /> |
There was a problem hiding this comment.
This production CLI dependency change has no corresponding entry under CHANGELOG.md's [Unreleased] section, so the project's release notes will omit the Sentry upgrade. Add a concise Changed or Security entry as required by the repository handoff protocol.
AGENTS.md reference: AGENTS.md:L107-L110
Useful? React with 👍 / 👎.
|
Updated Sentry from 6.6.0 to 6.8.0.
Release notes
Sourced from Sentry's releases.
6.8.0
Features ✨
Logs
log4netintegration by @Flash0ver in #5172NLogintegration by @Flash0ver in #5176Other
AddToTransactionsonSentryAttachment#5182 by @jamescrosswell in #5182SentrySdk.RecordTransactionto record already-completed transactions and spans (e.g. replayed through a proxy) #5333 by @jamescrosswell in #5333Environmentset on theScopenow gets synchronized to the native layers (sentry-cocoaandsentry-native) by @bitsandfoxes in #5365Fixes 🐛
SentrySpanProcessorno longer leaks spans whose Activity never ends (e.g. aborted requests); the Activity is now held via aWeakReferenceso orphaned spans are pruned once it is garbage-collected. by @Ermabo in #5393Dependencies ⬆️
Deps
Other
6.7.0
Features ✨
IgnoreTransactionsoption to filter out transactions by name, matching substrings or regular expressions against the transaction name (#5377) by @Adham-Kiwan in #5377SentryEventExtensions.IsFromUnhandledException) and terminal exceptions (SentryEventExtensions.IsFromTerminalException) by @jamescrosswell in #5177Fixes 🐛
Dependencies ⬆️
Deps
Other
Sentry.Extensions.LoggingFilters by @Flash0ver in #5297Commits viewable in compare view.
Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting
@dependabot rebase.Dependabot commands and options
You can trigger Dependabot actions by commenting on this PR:
@dependabot rebasewill rebase this PR@dependabot recreatewill recreate this PR, overwriting any edits that have been made to it@dependabot show <dependency name> ignore conditionswill show all of the ignore conditions of the specified dependency@dependabot ignore this major versionwill close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)@dependabot ignore this minor versionwill close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)@dependabot ignore this dependencywill close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)Note
Bump Sentry dependency from 6.6.0 to 6.8.0 in DINOForge.Tools.Cli
Updates the
SentryPackageReferencein DINOForge.Tools.Cli.csproj and refreshes packages.lock.json with updated hashes for Sentry and its transitive dependencies.Macroscope summarized df6a431.