Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[chore]: enable error-is-as rule from testifylint #34995

Merged
merged 1 commit into from
Sep 5, 2024

Conversation

mmorel-35
Copy link
Contributor

Description

Testifylint is a linter that provides best practices with the use of testify.

This PR enables error-is-as rule from testifylint

@mmorel-35 mmorel-35 marked this pull request as draft September 4, 2024 06:41
@mmorel-35 mmorel-35 marked this pull request as ready for review September 4, 2024 08:34
@songy23 songy23 added the ready to merge Code review completed; ready to merge by maintainers label Sep 4, 2024
@mmorel-35 mmorel-35 force-pushed the testifylint/error-is-as branch 2 times, most recently from 6b14793 to 6603893 Compare September 4, 2024 21:27
@github-actions github-actions bot added the cmd/otelcontribcol otelcontribcol command label Sep 4, 2024
cmd/otelcontribcol/go.mod Outdated Show resolved Hide resolved
cmd/otelcontribcol/go.sum Outdated Show resolved Hide resolved
Signed-off-by: Matthieu MOREL <matthieu.morel35@gmail.com>
@codeboten codeboten merged commit 0055e6e into open-telemetry:main Sep 5, 2024
156 checks passed
@github-actions github-actions bot added this to the next release milestone Sep 5, 2024
@mmorel-35 mmorel-35 deleted the testifylint/error-is-as branch September 5, 2024 16:23
f7o pushed a commit to f7o/opentelemetry-collector-contrib that referenced this pull request Sep 12, 2024
#### Description

Testifylint is a linter that provides best practices with the use of
testify.

This PR enables
[error-is-as](https://github.com/Antonboom/testifylint?tab=readme-ov-file#error-is-as)
rule from [testifylint](https://github.com/Antonboom/testifylint)

Signed-off-by: Matthieu MOREL <matthieu.morel35@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants