Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix: CLM improvements #554

Merged
merged 1 commit into from
Aug 18, 2022
Merged

Fix: CLM improvements #554

merged 1 commit into from
Aug 18, 2022

Conversation

nr-swilloughby
Copy link
Contributor

Fixes issue #551 by changing the single string value for Config.CodeLevelMetrics.IgnoredPrefix to the []string value Config.CodeLevelMetrics.IgnoredPrefixes. Adjusts associated option-setting functions to accept multiple string parameters. Deprecates the IgnoredPrefix field but uses its value if one is set to avoid breaking existing code.

Fixes issue #550 by correcting errors in package documentation.

@nr-swilloughby nr-swilloughby self-assigned this Aug 17, 2022
@nr-swilloughby nr-swilloughby added this to Awaiting User Input in Go Engineering Board via automation Aug 17, 2022
@nr-swilloughby nr-swilloughby marked this pull request as ready for review August 18, 2022 15:43
@nr-swilloughby nr-swilloughby merged commit 48c28dc into newrelic:develop Aug 18, 2022
Go Engineering Board automation moved this from Awaiting User Input to Done Aug 18, 2022
@nr-swilloughby nr-swilloughby deleted the clm_tweaks branch August 18, 2022 18:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

None yet

2 participants