Skip to content

[Linter] Add golang linter #3501

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

Open
wants to merge 12 commits into
base: main
Choose a base branch
from
Open

[Linter] Add golang linter #3501

wants to merge 12 commits into from

Conversation

jkuradobery
Copy link
Collaborator

No description provided.

Copy link
Contributor

Note

This is an automated comment that will be appended during run.

🟢 linux-x86_64-relwithdebinfo: all tests PASSED for commit 42c0d48.

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
8490 8490 0 0 0 0

Copy link
Contributor

Note

This is an automated comment that will be appended during run.

🟢 linux-x86_64-relwithdebinfo: all tests PASSED for commit 7ea3ab9.

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
8490 8490 0 0 0 0

Copy link
Contributor

Note

This is an automated comment that will be appended during run.

🟢 linux-x86_64-relwithdebinfo: all tests PASSED for commit 090f456.

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
8490 8490 0 0 0 0

contextcheck - Checks for context propagation in all function if parent function uses context. There is no point to handle context in functions computing values.
cyclop - really hard to measure function complexity, should avoid trying
decorder - we do not have a strong opinion regarding this
depguard - build is performed using ya.make, so, pointless.
dupl - noisy
exhaustive - noisy
gocognit - also strange metric
gocyclo - strange metric
godox - we need todos
gomodguard - we build w ya.make
grouper - do not see the point
iface - noisy
Copy link
Contributor

github-actions bot commented May 16, 2025

Note

This is an automated comment that will be appended during run.

🔴 linux-x86_64-relwithdebinfo: some tests FAILED for commit fece863.

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
8490 8469 0 6 13 2

🟢 linux-x86_64-relwithdebinfo: all tests PASSED for commit fece863.

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
191 191 0 0 0 0

Copy link
Contributor

github-actions bot commented May 22, 2025

Note

This is an automated comment that will be appended during run.

🔴 linux-x86_64-relwithdebinfo: some tests FAILED for commit 6cc2624.

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
8490 8489 0 1 0 0

🟢 linux-x86_64-relwithdebinfo: all tests PASSED for commit 6cc2624.

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
43 43 0 0 0 0

Copy link
Contributor

github-actions bot commented May 30, 2025

Note

This is an automated comment that will be appended during run.

🟢 linux-x86_64-relwithdebinfo: all tests PASSED for commit 6718e1e.

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
8490 8487 0 0 1 2

🟢 linux-x86_64-relwithdebinfo: all tests PASSED for commit 6718e1e.

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
112 112 0 0 0 0

Copy link
Contributor

github-actions bot commented Jun 2, 2025

Note

This is an automated comment that will be appended during run.

🟢 linux-x86_64-relwithdebinfo: all tests PASSED for commit 854b6e7.

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
8490 8490 0 0 0 0

Copy link
Contributor

github-actions bot commented Jun 4, 2025

Note

This is an automated comment that will be appended during run.

🟢 linux-x86_64-relwithdebinfo: all tests PASSED for commit 6c7d531.

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
8490 8490 0 0 0 0

Copy link
Contributor

Note

This is an automated comment that will be appended during run.

🟢 linux-x86_64-relwithdebinfo: all tests PASSED for commit 70fbe36.

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
8490 8490 0 0 0 0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant