Skip to content

fix config - #669

Merged
cleverchuk merged 1 commit into
mainfrom
cc/NH-153862
Nov 5, 2026
Merged

cleverchuk merged 1 commit into
mainfrom
cc/NH-153862

Conversation

@cleverchuk

Copy link
Copy Markdown
Contributor

@cleverchuk
cleverchuk requested review from a team as code owners November 5, 2026 17:41
Copilot AI balanced review requested due to automatic review settings November 5, 2026 17:41

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🟢 Approval recommended

No active review issues are recorded for the Java instrumentation configuration-key correction.

No open issues 🎉

What changed in this PR

Corrects the Java instrumentation configuration key from default-enabled to default_enabled across the example configuration and test fixtures.

Changes:

  • Aligns all four YAML files on instrumentation/development.java.common.default_enabled.
  • Preserves the configured value of true.
File Description
smoke-tests/​sdk-config.yaml Corrects the key used by smoke tests.
sdk-config-kitchen-sink.yaml Corrects the key in the comprehensive configuration example.
libs/​shared/​src/​test/​resources/​sdk-config.yaml Updates the shared-library test fixture.
custom/​src/​test/​resources/​sdk-config.yaml Updates the custom-module test fixture.

Review effort: Balanced

@cleverchuk
cleverchuk merged commit 333067c into main Nov 5, 2026
10 checks passed
@cleverchuk
cleverchuk deleted the cc/NH-153862 branch November 5, 2026 17:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants