Skip to content

fix: omit comparison to bool constant#479

Merged
lbajolet-hashicorp merged 1 commit intomainfrom
fix/omit-comparison-to-bool-constant
Oct 16, 2024
Merged

fix: omit comparison to bool constant#479
lbajolet-hashicorp merged 1 commit intomainfrom
fix/omit-comparison-to-bool-constant

Conversation

@tenthirtyam
Copy link
Copy Markdown
Collaborator

Summary

Simplifies the condition for checking if the image import request was created.

➜ staticcheck builder/vsphere/supervisor/step_import_image.go                                             
builder/vsphere/supervisor/step_import_image.go:166:69: should omit comparison to bool constant, can be simplified to !v.(bool) (S1002)
➜ staticcheck builder/vsphere/supervisor/step_import_image.go                                             
➜

Omit comparison to bool constant.

Signed-off-by: Ryan Johnson <ryan.johnson@broadcom.com>
@tenthirtyam tenthirtyam added the technical-debt Technical Debt label Oct 16, 2024
@tenthirtyam tenthirtyam added this to the v1.4.1 milestone Oct 16, 2024
@tenthirtyam tenthirtyam self-assigned this Oct 16, 2024
@tenthirtyam tenthirtyam requested a review from a team as a code owner October 16, 2024 00:25
Copy link
Copy Markdown
Contributor

@lbajolet-hashicorp lbajolet-hashicorp left a comment

Choose a reason for hiding this comment

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

LGTM!

@lbajolet-hashicorp lbajolet-hashicorp merged commit 58d9b55 into main Oct 16, 2024
@lbajolet-hashicorp lbajolet-hashicorp deleted the fix/omit-comparison-to-bool-constant branch October 16, 2024 19:52
@github-actions
Copy link
Copy Markdown

I'm going to lock this pull request because it has been closed for 30 days. This helps our maintainers find and focus on the active issues.

If you have found a problem that seems related to this change, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jan 30, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

technical-debt Technical Debt

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants