Skip to content

refactor: simplify tests with require.ErrorContains#5855

Merged
caarlos0 merged 1 commit intogoreleaser:mainfrom
alexandear-org:refactor/require-error-contains
Jun 25, 2025
Merged

refactor: simplify tests with require.ErrorContains#5855
caarlos0 merged 1 commit intogoreleaser:mainfrom
alexandear-org:refactor/require-error-contains

Conversation

@alexandear
Copy link
Copy Markdown
Contributor

require.Error(t, err) is not needed because require.ErrorContains fails if err is nil or err.Error() is empty.

@pull-request-size pull-request-size Bot added the size/S Denotes a PR that changes 10-29 lines, ignoring generated files. label Jun 25, 2025
Copy link
Copy Markdown
Member

@caarlos0 caarlos0 left a comment

Choose a reason for hiding this comment

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

nice, thank you!

@caarlos0 caarlos0 merged commit 39e7191 into goreleaser:main Jun 25, 2025
12 of 14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size/S Denotes a PR that changes 10-29 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants