-
Notifications
You must be signed in to change notification settings - Fork 1.6k
🌱 (chore): enable ginkgolinter forbid-spec-pollution to enforce isolated test specs #4710
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 ginkgolinter forbid-spec-pollution to enforce isolated test specs #4710
Conversation
Hi @kersten. Thanks for your PR. I'm waiting for a kubernetes-sigs member to verify that this patch is reasonable to test. If it is, they should reply with Once the patch is verified, the new status will be reflected by the I understand the commands that are listed here. Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. |
/ok-to-test |
/lgtm cancel See that this one is falling in the linter checks: https://github.com/kubernetes-sigs/kubebuilder/actions/runs/14126922651/job/39578281565?pr=4710 |
Hey @camilamacedo86, we should wait for the other PRs to get merged, then I will rebase and the test should be green again 👍🏻 |
1544ba3
to
b6922e7
Compare
b6922e7
to
2b96991
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It seems to make sense to add this.
I'm fine moving forward with this one. Let's merge it and see if we encounter any issues down the line.
/ok-to-test
/lgtm
/approve
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: camilamacedo86, kersten The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
🌱 Enable ginkgolinter forbid-spec-pollution
This PR adds the following configuration to
.golangci.yml
:Enabling forbid-spec-pollution ensures all shared test state is scoped within BeforeEach or similar blocks, helping prevent flaky tests and improving test isolation in Ginkgo test suites.