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

Address findings from the linters configured in golangci-lint. #457

Open
3 of 4 tasks
tenthirtyam opened this issue Nov 15, 2024 · 0 comments
Open
3 of 4 tasks

Address findings from the linters configured in golangci-lint. #457

tenthirtyam opened this issue Nov 15, 2024 · 0 comments
Labels
github-actions GitHub Actions technical-debt Technical Debt
Milestone

Comments

@tenthirtyam
Copy link
Contributor

Code of Conduct

  • I have read and agree to the Code of Conduct.
  • Vote on this issue by adding a 👍 reaction to the original issue initial description to help the maintainers prioritize.
  • Do not leave "+1" or other comments that do not add relevant information or questions.
  • If you are interested in working on this issue or have submitted a pull request, please leave a comment.

Terraform

latest

Terraform Provider

latest

Description

After merging #435 to update the golangci-lint workflow the action currently does not complete successfully as there is some technical debt that needs to be addressed based on the findings by a few of the linters.

You can expand the results here or run golangci-lint run in the project root locally to observe the findings. These include:

  • integer overflow conversion int
  • const example is unused
  • import example is not allowed from list 'Main'
  • naked return in function
  • append only allowed to cuddle with appended value
  • only one cuddle assignment allowed before if statement
  • expressions should not be cuddled with blocks
  • if statements should only be cuddled with assignments

Affected Resources or Data Sources

Not applicable.

Terraform Configuration

Not applicable.

Debug Output

Not applicable.

Panic Output

Not applicable.

Expected Behavior

The GitHub Action for the run of golangci-lint run should find no error

Actual Behavior

You can expand the results here or run golangci-lint run in the project root locally to observe the findings. These include:

  • integer overflow conversion int
  • const example is unused
  • import example is not allowed from list 'Main'
  • naked return in function
  • append only allowed to cuddle with appended value
  • only one cuddle assignment allowed before if statement
  • expressions should not be cuddled with blocks
  • if statements should only be cuddled with assignments

Steps to Reproduce

Run golangci-lint run in the root of the project.

Environment Details

No response

Screenshots

No response

References

GitHub Action Run Results

@tenthirtyam tenthirtyam added github-actions GitHub Actions technical-debt Technical Debt labels Nov 15, 2024
@tenthirtyam tenthirtyam added this to the On Deck milestone Nov 15, 2024
@github-actions github-actions bot added the pending-review Pending Review label Nov 15, 2024
@tenthirtyam tenthirtyam removed the pending-review Pending Review label Nov 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
github-actions GitHub Actions technical-debt Technical Debt
Projects
None yet
Development

No branches or pull requests

1 participant