Skip to content

Pull requests: go-critic/go-critic

New pull request New
Clear current search query, filters, and sorts
9 Open 178 Closed
9 Open 178 Closed
Author
Filter by author
Loading
Label
Filter by label
Loading
Use alt + click/return to exclude labels
or ⇧ + click/return for logical OR
Projects
Filter by project
Loading
Milestones
Filter by milestone
Loading
Reviews
Filter by reviews
No reviews Review required Approved review Changes requested
Assignee
Filter by who’s assigned
Assigned to nobody Loading
Sort
Sort by
Newest Oldest Most commented Least commented Recently updated Least recently updated Best match
Most reactions
👍 👎 😄 🎉 😕 ❤️ 🚀 👀

Pull requests list

Added colored output, json output. And removed unnecessary log prefixes.
#603 by AnikHasibul was closed Sep 6, 2018 Loading…
20
Quasilyte/type guard v1
#19 by quasilyte was merged May 9, 2018 Loading…
19
lint: add checker for unexported method calls on unexported types
#102 by fexolm was merged May 26, 2018 Loading…
1
12
lint: add simple parenthesis check
#22 by fexolm was merged May 8, 2018 Loading…
11
lint/lenNil: detect redundant x!=nil before len(x)
#293 by cristaloleg was closed Jun 28, 2018 Loading…
11
lint/param-duplication: get correct ast node
#137 by fexolm was merged Jun 4, 2018 Loading…
10
lint: quickfix implementation [doesn't work for now] Don't merge help wanted Extra attention is needed question Further information is requested
#160 by fexolm was closed Jun 8, 2018 Loading…
10
lint: add param-name checker
#7 by quasilyte was merged May 6, 2018 Loading…
8
lint: add checks for uninformative comments
#115 by weeellz was merged May 26, 2018 Loading…
8
lint: add stddef checker v1 Don't merge
#78 by quasilyte was closed May 31, 2018 Loading…
7
docs/contributing.md: add issues guide.
#142 by fexolm was merged Jun 5, 2018 Loading…
1
6
docs/overview: use html tables instead of md tables
#156 by quasilyte was merged Jun 7, 2018 Loading…
5
lint: make longChain more conservative Don't merge
#466 by quasilyte was closed Dec 5, 2018 Loading…
1
5
implemented caseOrder checker for values
#1082 by peakle was closed Dec 26, 2021 Loading…
5
add builtin shadow checker documentation
#128 by bontequero was merged Jun 3, 2018 Loading…
4
cmd/kfulint: add end2end test runner
#50 by quasilyte was merged May 10, 2018 Loading…
4
lint/yodaStyleExpr: add support for ops: < > >= <=
#356 by cristaloleg was closed Jul 14, 2018 Loading…
4
rich config format (#574)
#585 by ewgRa was closed Dec 5, 2018 Loading…
4
checkers/httpNoBody: also warn on using "nil" instead of "http.NoBody" on (net/http/httptest).NewRequest
#1345 by leonklingele was merged May 19, 2023 Loading…
4
checker: add zeroByteRepeat for avoiding bytes.Repeat([]byte{0}, x)
#1488 by codeimmortal was merged May 13, 2025 Loading…
4
Use proper grammar
#666 by mikechernev was closed Oct 2, 2018 Loading…
3
all: config for checkers
#374 by cristaloleg was closed Jul 11, 2018 Loading…
3
[WIP] lint: implementation assert in code files Don't merge
#236 by cristaloleg was closed Jun 23, 2018 Loading…
3
lint: replace make([]T, 0) slice init to []T{}
#705 by quasilyte was merged Oct 29, 2018 Loading…
@fexolm
3
lint: add range-expr-copy checker
#122 by quasilyte was merged Jun 3, 2018 Loading…
3
ProTip! Type g i on any issue or pull request to go back to the issue listing page.