Skip to content

Commit 18ed8fe

Browse files
authored
[chore] Use mdatagen for configopaque and configtelemetry (#15696)
<!--Ex. Fixing a bug - Describe the bug and how this fixes the issue. Ex. Adding a feature - Explain what this achieves.--> #### Description Completes the mdatagen migration for the `config/` packages. `configopaque` and `configtelemetry` were the only two left using a hand-written `package_test.go` after #15639. <!-- Issue number if applicable --> #### Link to tracking issue No dedicated tracking issue this follows the same pattern established in #15632 and #15639. Happy to open a tracking issue first if maintainers would prefer that. <!--Describe what testing was performed and which tests were added.--> #### Testing `go test -count=1 ./...` passes for both `config/configopaque` and `config/configtelemetry`. <!--Authorship attestation. See AGENTS.md for details. AI agents must not check this box on behalf of the user; the human author must check it themselves before the PR is ready for review.--> #### Authorship - [x] I, a human, wrote this pull request description myself. <!--Please delete paragraphs that you did not use before submitting.-->
1 parent 0ffbf34 commit 18ed8fe

4 files changed

Lines changed: 6 additions & 4 deletions

File tree

config/configopaque/doc.go

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -19,3 +19,5 @@
1919
// or become widely used in the Go ecosystem, these will eventually be implemented
2020
// by configopaque.String as well. This is not considered a breaking change.
2121
package configopaque // import "go.opentelemetry.io/collector/config/configopaque"
22+
23+
//go:generate mdatagen metadata.yaml

config/configopaque/package_test.go renamed to config/configopaque/generated_package_test.go

Lines changed: 1 addition & 2 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

config/configtelemetry/doc.go

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -40,3 +40,5 @@
4040
// The signals associated with this level may exhibit high cardinality, high data volume, or high
4141
// computational cost.
4242
package configtelemetry // import "go.opentelemetry.io/collector/config/configtelemetry"
43+
44+
//go:generate mdatagen metadata.yaml

config/configtelemetry/package_test.go renamed to config/configtelemetry/generated_package_test.go

Lines changed: 1 addition & 2 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

0 commit comments

Comments
 (0)