Skip to content

Conversation

@tenthirtyam
Copy link
Collaborator

Error messages should not begin with capitalization or end with punctuation.

@tenthirtyam tenthirtyam added this to the v1.5.0 milestone Dec 1, 2024
@tenthirtyam tenthirtyam self-assigned this Dec 1, 2024
@tenthirtyam tenthirtyam requested a review from a team as a code owner December 1, 2024 19:56
Copy link
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.

Left a few nits on some comments, mostly to steer users towards a resolution in some cases, but besides that this LGTM!

Error messages should not begin with capitalization or end with punctuation.

Signed-off-by: Ryan Johnson <[email protected]>
@tenthirtyam
Copy link
Collaborator Author

@lbajolet-hashicorp - ready for review again.

Copy link
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, thanks for the reroll @tenthirtyam!

@lbajolet-hashicorp lbajolet-hashicorp merged commit c038f87 into main Jan 27, 2025
14 checks passed
@lbajolet-hashicorp lbajolet-hashicorp deleted the chore/error-msgs branch January 27, 2025 17:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants