Skip to content
This repository has been archived by the owner on Jan 8, 2024. It is now read-only.

cli: more specific error msgs on install #1004

Merged
merged 2 commits into from
Jan 22, 2021
Merged

cli: more specific error msgs on install #1004

merged 2 commits into from
Jan 22, 2021

Conversation

krantzinator
Copy link
Contributor

Small PR to match the error messaging we give on uninstall if the -platform flag is missing.

@krantzinator krantzinator requested a review from a team January 21, 2021 22:45
@github-actions github-actions bot added the core label Jan 21, 2021
briancain
briancain suggested changes Jan 21, 2021
View reviewed changes
internal/cli/install.go Outdated Show resolved Hide resolved
@krantzinator @briancain
Update internal/cli/install.go
b14bcae 
Co-authored-by: Brian Cain <bcain@hashicorp.com>
briancain
briancain approved these changes Jan 22, 2021
View reviewed changes
Copy link
Member

@briancain briancain left a comment

Choose a reason for hiding this comment

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

woohoo! 👍 thanks for the update 😄

@krantzinator krantzinator merged commit d644c0d into main Jan 22, 2021
@krantzinator krantzinator deleted the e-install-platform-error-msg branch January 22, 2021 18:44
@hc-waypoint-bot hc-waypoint-bot mentioned this pull request Jan 22, 2021
Merged
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@mitchellh mitchellh mitchellh approved these changes

@briancain briancain briancain approved these changes

Assignees
No one assigned
Labels
core
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@krantzinator @mitchellh @briancain