Skip to content

refactor: replace type assertion on errors#314

Merged
tenthirtyam merged 1 commit intomainfrom
refactor/replaces-type-assertion
May 21, 2025
Merged

refactor: replace type assertion on errors#314
tenthirtyam merged 1 commit intomainfrom
refactor/replaces-type-assertion

Conversation

@tenthirtyam
Copy link
Copy Markdown
Collaborator

Replaces type assertion on errors with a call to errors.As.

The preferred way for checking for a specific error type is to use the errors.As function from the standard library as this function traverses the chain of the wrapped errors while checking for a specific error type.

~/Downloads/packer-plugin-vmware git:[refactor/replaces-type-assertion]
go fmt ./...

~/Downloads/packer-plugin-vmware git:[refactor/replaces-type-assertion]
make dev
packer plugins install --path packer-plugin-vmware "github.com/hashicorp/vmware"
Successfully installed plugin github.com/hashicorp/vmware from /Users/johnsonryan/Downloads/packer-plugin-vmware/packer-plugin-vmware to /Users/johnsonryan/.packer.d/plugins/github.com/hashicorp/vmware/packer-plugin-vmware_v1.1.1-dev_x5.0_darwin_amd64

~/Downloads/packer-plugin-vmware git:[refactor/replaces-type-assertion]
make build

~/Downloads/packer-plugin-vmware git:[refactor/replaces-type-assertion]
make test
?       github.com/hashicorp/packer-plugin-vmware       [no test files]
ok      github.com/hashicorp/packer-plugin-vmware/builder/vmware/common 6.855s
ok      github.com/hashicorp/packer-plugin-vmware/builder/vmware/iso    2.109s
ok      github.com/hashicorp/packer-plugin-vmware/builder/vmware/vmx    2.617s
?       github.com/hashicorp/packer-plugin-vmware/version       [no test files]

~/Downloads/packer-plugin-vmware git:[refactor/replaces-type-assertion]
golangci-lint run
0 issues.

Replaces type assertion on errors with a call to `errors.As`.

The preferred way for checking for a specific error type is to use the `errors.As` function from the standard library as this function traverses the chain of the wrapped errors while checking for a specific error type.

Signed-off-by: Ryan Johnson <ryan.johnson@broadcom.com>
@tenthirtyam tenthirtyam added this to the v1.2.0 milestone May 21, 2025
@tenthirtyam tenthirtyam requested a review from JenGoldstrich May 21, 2025 20:28
@tenthirtyam tenthirtyam self-assigned this May 21, 2025
@tenthirtyam tenthirtyam requested a review from a team as a code owner May 21, 2025 20:28
@tenthirtyam tenthirtyam merged commit 5bd1900 into main May 21, 2025
14 checks passed
@tenthirtyam tenthirtyam deleted the refactor/replaces-type-assertion branch May 21, 2025 22:48
@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

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants