Skip to content

fix(.golangci.yml): Configure staticcheck to ignore ginkgo/gomega dot-imports#973

Merged
openshift-merge-bot[bot] merged 1 commit into
redhat-developer:masterfrom
olivergondza:lint-ignore-dotimport-in-tests
Sep 23, 2025
Merged

fix(.golangci.yml): Configure staticcheck to ignore ginkgo/gomega dot-imports#973
openshift-merge-bot[bot] merged 1 commit into
redhat-developer:masterfrom
olivergondza:lint-ignore-dotimport-in-tests

Conversation

@olivergondza

Copy link
Copy Markdown
Collaborator

Ignore dot imports for e2e test helpers

What type of PR is this?

Uncomment only one /kind line, and delete the rest.
For example, > /kind bug would simply become: /kind bug

/kind bug
/kind cleanup
/kind failing-test
/kind enhancement
/kind documentation
/kind code-refactoring

What does this PR do / why we need it:

Ginkgo tests uses dot-imports that are reported as undesirable:

	. "github.com/onsi/ginkgo/v2"
	. "github.com/onsi/gomega"

Have you updated the necessary documentation?

  • [no] Documentation update is required by this PR.
  • [no] Documentation has been updated.

Which issue(s) this PR fixes:

None

Test acceptance criteria:

  • [no] Unit Test
  • [no] E2E Test

How to test changes / Special notes to the reviewer:

Same fix in argocd-operator verified already: argoproj-labs/argocd-operator#1880

…-imports

Signed-off-by: Oliver Gondža <ogondza@gmail.com>
@openshift-ci openshift-ci Bot added the kind/failing-test Categorizes issue or PR as related to a frequently failing test. label Sep 23, 2025
@openshift-ci openshift-ci Bot requested review from chetan-rns and jgwest September 23, 2025 12:30
@anandf

anandf commented Sep 23, 2025

Copy link
Copy Markdown
Member

/lgtm

@anandf

anandf commented Sep 23, 2025

Copy link
Copy Markdown
Member

/approve

@openshift-ci

openshift-ci Bot commented Sep 23, 2025

Copy link
Copy Markdown

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: anandf

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@jgwest jgwest left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM too, thanks @olivergondza! This is a much better solution for this then what I came up with.

(Unfortunately, if you're using VS Code, you may still receive warnings for these in Problems panel, but that's a separate issue)

@openshift-merge-bot openshift-merge-bot Bot merged commit 78c59ae into redhat-developer:master Sep 23, 2025
16 checks passed
@olivergondza olivergondza deleted the lint-ignore-dotimport-in-tests branch September 24, 2025 06:34
olivergondza added a commit to olivergondza/gitops-operator that referenced this pull request Feb 13, 2026
Not needed after redhat-developer#973

Signed-off-by: Oliver Gondža <ogondza@gmail.com>
jgwest pushed a commit that referenced this pull request Feb 18, 2026
Not needed after #973

Signed-off-by: Oliver Gondža <ogondza@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved kind/failing-test Categorizes issue or PR as related to a frequently failing test. lgtm

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants