Skip to content

Conversation

@richardelms
Copy link
Contributor

@richardelms richardelms commented Nov 11, 2025

This pull request refactors the configuration option for specifying which release stages should send errors to Bugsnag. The property and related methods have been renamed from notifyReleaseStages to enabledReleaseStages throughout the codebase, including classes, methods, documentation, tests, and configuration files. This change improves clarity and consistency in naming, making it easier to understand the intent of the setting.

API and Configuration Refactor

  • Renamed the configuration property in Configuration from notifyReleaseStages to enabledReleaseStages, and updated all related logic to use the new name. [1] [2]
  • Updated the Bugsnag and BugsnagAppender classes: methods, fields, and documentation now use enabledReleaseStages instead of notifyReleaseStages. Deprecated old setter methods and added new ones for the updated naming. [1] [2] [3] [4] [5] [6] [7]

Test and Scenario Updates

  • Refactored all tests and scenario classes to use enabledReleaseStages instead of notifyReleaseStages, including method names, assertions, and comments. [1] [2] [3] [4] [5] [6] [7] [8] [9]

Configuration File Changes

  • Updated XML configuration files to use <enabledReleaseStage> tags instead of <notifyReleaseStage>. [1] [2] [3]

@richardelms richardelms self-assigned this Nov 11, 2025
@richardelms richardelms marked this pull request as ready for review November 11, 2025 14:57
@richardelms richardelms requested a review from lemnik November 13, 2025 09:11
Copy link
Contributor

@lemnik lemnik left a comment

Choose a reason for hiding this comment

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

LGTM

@richardelms richardelms merged commit b530c39 into integration/uplift-2025 Nov 13, 2025
1 check passed
@richardelms richardelms deleted the enabledReleaseStages branch November 13, 2025 09:59
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