Skip to content

Conversation

@Litarnus
Copy link
Contributor

@Litarnus Litarnus commented Dec 2, 2025

No description provided.

@Litarnus Litarnus self-assigned this Dec 2, 2025
@stayallive stayallive enabled auto-merge (squash) December 2, 2025 10:36
@stayallive stayallive merged commit 0d6b477 into master Dec 2, 2025
37 checks passed
@stayallive stayallive deleted the Prepare-4.20.0 branch December 2, 2025 10:36
# Changelog

## 4.20.0

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Bug: The SDK_VERSION constant in src/Sentry/Laravel/Version.php is not updated to 4.20.0.
Severity: CRITICAL | Confidence: High

🔍 Detailed Analysis

The SDK_VERSION constant in src/Sentry/Laravel/Version.php remains '4.19.0' despite the CHANGELOG.md announcing version 4.20.0. This mismatch causes the SDK to report an incorrect version (4.19.0) to Sentry for all events and when users run php artisan about. This leads to inaccurate version tracking and diagnostics, impacting error tracking and release management.

💡 Suggested Fix

Update public const SDK_VERSION to '4.20.0' in src/Sentry/Laravel/Version.php as part of the release preparation commit.

🤖 Prompt for AI Agent
Review the code at the location below. A potential bug has been identified by an AI
agent.
Verify if this is a real issue. If it is, propose a fix; if not, explain why it's not
valid.

Location: CHANGELOG.md#L4

Potential issue: The `SDK_VERSION` constant in `src/Sentry/Laravel/Version.php` remains
`'4.19.0'` despite the `CHANGELOG.md` announcing version `4.20.0`. This mismatch causes
the SDK to report an incorrect version (`4.19.0`) to Sentry for all events and when
users run `php artisan about`. This leads to inaccurate version tracking and
diagnostics, impacting error tracking and release management.

Did we get this right? 👍 / 👎 to inform future reviews.
Reference ID: 4809394

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants