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

[chore] add wastedassign linter #26522

Merged
merged 1 commit into from
Sep 8, 2023

Conversation

atoulme
Copy link
Contributor

@atoulme atoulme commented Sep 7, 2023

See #25060 for original request.

This adds the wastedassign linter with default configuration. It ensures no assignmnent is made to a variable and then not used.

clusterNameSet = projectCfg.includesByClusterName
// exclude is initialized
case len(exclude) > 0 && len(include) == 0:
whitelist = false
allowed = false
Copy link
Contributor

Choose a reason for hiding this comment

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

this isn't needed is it?

@codeboten codeboten merged commit 788f03e into open-telemetry:main Sep 8, 2023
90 checks passed
@github-actions github-actions bot added this to the next release milestone Sep 8, 2023
@atoulme atoulme deleted the wastedassign_linter branch September 8, 2023 22:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants