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.

Sign up for GitHub

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

Jump to bottom

[discovery] Don't use component.MustNewIDWithName #4565

Merged

dmitryax merged 1 commit into main from dont-use-must

Mar 31, 2024

Contributor

dmitryax commented Mar 31, 2024

To avoid collector crashes. We don't control the input in most of the cases


          [discovery] Don't use component.MustNewIDWithName

7b79b7f

To avoid collector crashes. We don't control the input in most of the cases

dmitryax requested review from a team as code owners

March 31, 2024 04:03

atoulme approved these changes

View reviewed changes

dmitryax merged commit abf230b into main

27 checks passed

dmitryax deleted the dont-use-must branch

March 31, 2024 04:39

github-actions bot locked and limited conversation to collaborators

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