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.

@atoulme atoulme force-pushed the wastedassign_linter branch from 16a5a53 to 4beb1ff Compare September 8, 2023 06:08
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?

@atoulme atoulme force-pushed the wastedassign_linter branch from 7f4a30f to 29cb2db Compare September 8, 2023 19:16
@codeboten codeboten merged commit 788f03e into open-telemetry:main Sep 8, 2023
@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