Ecosyste.ms: Issues

An open API service for providing issue and pull request metadata for open source projects.

GitHub / go-critic/go-critic issues and pull requests

#1344 - httpNoBody: also warn on using "nil" instead of "http.NoBody" on (net/http/httptest).NewRequest

Issue - State: closed - Opened by leonklingele over 1 year ago - 3 comments
Labels: checker stabilization

#1343 - build(deps): bump golang.org/x/tools from 0.9.0 to 0.9.1

Pull Request - State: closed - Opened by dependabot[bot] over 1 year ago
Labels: dependencies, go

#1342 - build(deps): bump golang.org/x/tools from 0.8.0 to 0.9.0

Pull Request - State: closed - Opened by dependabot[bot] over 1 year ago
Labels: dependencies, go

#1341 - new check: mismatching format

Issue - State: open - Opened by maddin200 almost 2 years ago - 1 comment
Labels: checker request

#1340 - checkers: fix setting of Go version for ruleguard

Pull Request - State: closed - Opened by alexandear almost 2 years ago - 1 comment

#1339 - checkers: remove err.Error redundantChecker suggest

Pull Request - State: closed - Opened by alexandear almost 2 years ago - 3 comments

#1338 - redundantSprint false positive with err.Error() when err is nil

Issue - State: closed - Opened by alexandear almost 2 years ago - 1 comment

#1337 - checkers: empty slice

Pull Request - State: open - Opened by demoManito almost 2 years ago

#1336 - build(deps): bump golang.org/x/tools from 0.7.0 to 0.8.0

Pull Request - State: closed - Opened by dependabot[bot] almost 2 years ago
Labels: dependencies, go

#1335 - checkers/octalLiteral: should warn only for Go >= 1.13

Pull Request - State: closed - Opened by alexandear almost 2 years ago - 1 comment

#1334 - checkers: add purego

Pull Request - State: open - Opened by cristaloleg almost 2 years ago

#1333 - all: use checker tags

Pull Request - State: closed - Opened by cristaloleg almost 2 years ago

#1332 - deps: bump actions/setup-go from 3 to 4

Pull Request - State: closed - Opened by dependabot[bot] almost 2 years ago
Labels: dependencies, github_actions

#1331 - deps: bump github.com/go-toolsmith/pkgload from 1.2.1 to 1.2.2

Pull Request - State: closed - Opened by dependabot[bot] almost 2 years ago
Labels: dependencies, go

#1330 - deps: bump golang.org/x/tools from 0.6.0 to 0.7.0

Pull Request - State: closed - Opened by dependabot[bot] almost 2 years ago
Labels: dependencies, go

#1329 - checkers/wrapperFunc: merge into master

Pull Request - State: closed - Opened by cristaloleg almost 2 years ago

#1328 - cmd/gocritic: add concurrency flag

Pull Request - State: closed - Opened by cristaloleg almost 2 years ago

#1327 - cmd/gocritic: use checkGenerated flag properly

Pull Request - State: closed - Opened by cristaloleg almost 2 years ago

#1326 - How to speed it up

Issue - State: closed - Opened by tbarbugli almost 2 years ago - 18 comments
Labels: question

#1325 - dev: better linter config

Pull Request - State: closed - Opened by cristaloleg almost 2 years ago

#1324 - ci: trigger more often, use more versions & do cron

Pull Request - State: closed - Opened by cristaloleg almost 2 years ago

#1323 - deps: bump github.com/cristalhq/acmd from 0.11.0 to 0.11.1

Pull Request - State: closed - Opened by dependabot[bot] almost 2 years ago
Labels: dependencies, go

#1322 - checkers: add sloppyTestFuncName

Pull Request - State: closed - Opened by cristaloleg almost 2 years ago

#1321 - all: get rid of framework package

Pull Request - State: closed - Opened by cristaloleg almost 2 years ago

#1320 - deps: bump toolmantim/release-drafter from 5.20.0 to 5.23.0

Pull Request - State: closed - Opened by dependabot[bot] almost 2 years ago
Labels: dependencies, github_actions

#1319 - checkers: run go/analysis concurrently

Pull Request - State: closed - Opened by cristaloleg almost 2 years ago

#1318 - cmd,framework: remove lintmain

Pull Request - State: closed - Opened by cristaloleg almost 2 years ago

#1317 - all: cleanup internal things

Pull Request - State: closed - Opened by cristaloleg almost 2 years ago

#1316 - deps: bump

Pull Request - State: closed - Opened by cristaloleg almost 2 years ago

#1315 - update gogrep and ruleguard deps

Pull Request - State: closed - Opened by quasilyte almost 2 years ago

#1314 - New release?

Issue - State: closed - Opened by ldez almost 2 years ago - 6 comments

#1313 - Ignore generated files or specific directory?

Issue - State: closed - Opened by tonsV2 almost 2 years ago - 2 comments
Labels: question

#1312 - deps: bump golang.org/x/tools from 0.5.0 to 0.6.0

Pull Request - State: closed - Opened by dependabot[bot] almost 2 years ago
Labels: dependencies, go

#1311 - Non zero exit code in case of issues

Issue - State: closed - Opened by tonsV2 almost 2 years ago

#1310 - Compilation error under Go 1.20

Issue - State: closed - Opened by mpldr almost 2 years ago - 3 comments

#1309 - checkers: fix hugeParam and rangeValCopy when generics

Pull Request - State: closed - Opened by cristaloleg almost 2 years ago

#1308 - deps: bump github.com/go-toolsmith/typep from 1.0.2 to 1.1.0

Pull Request - State: closed - Opened by dependabot[bot] almost 2 years ago
Labels: dependencies, go

#1307 - build(deps): bump github.com/go-toolsmith/astcopy from 1.0.2 to 1.1.0

Pull Request - State: closed - Opened by dependabot[bot] almost 2 years ago - 1 comment
Labels: dependencies, go

#1306 - deps: bump github.com/go-toolsmith/astcast from 1.0.0 to 1.1.0

Pull Request - State: closed - Opened by dependabot[bot] almost 2 years ago
Labels: dependencies, go

#1305 - deps: bump github.com/go-toolsmith/astp from 1.0.0 to 1.1.0

Pull Request - State: closed - Opened by dependabot[bot] almost 2 years ago
Labels: dependencies, go

#1304 - deps: bump github.com/go-toolsmith/astequal from 1.0.3 to 1.1.0

Pull Request - State: closed - Opened by dependabot[bot] almost 2 years ago
Labels: dependencies, go

#1303 - deps: bump github.com/go-toolsmith/astfmt from 1.0.0 to 1.1.0

Pull Request - State: closed - Opened by dependabot[bot] almost 2 years ago
Labels: dependencies, go

#1302 - go,ci: update Go version

Pull Request - State: closed - Opened by cristaloleg almost 2 years ago

#1301 - deps: bump

Pull Request - State: closed - Opened by cristaloleg almost 2 years ago - 1 comment

#1300 - Add check to avoid always nil can be returned

Issue - State: open - Opened by oneumyvakin almost 2 years ago - 2 comments
Labels: checker request

#1298 - Panic with `hugeParamChecker` with generic struct

Issue - State: closed - Opened by nknordeen about 2 years ago - 3 comments

#1297 - rangValCopy panics on code with generics

Issue - State: closed - Opened by ryancurrah about 2 years ago - 1 comment

#1296 - deps: bump golang.org/x/tools from 0.3.0 to 0.5.0

Pull Request - State: closed - Opened by dependabot[bot] about 2 years ago
Labels: dependencies, go

#1295 - build(deps): bump toolmantim/release-drafter from 5.20.0 to 5.22.0

Pull Request - State: closed - Opened by dependabot[bot] about 2 years ago - 1 comment
Labels: dependencies, github_actions

#1294 - build(deps): bump github.com/quasilyte/go-ruleguard/dsl from 0.3.21 to 0.3.22

Pull Request - State: closed - Opened by dependabot[bot] about 2 years ago - 1 comment
Labels: dependencies, go

#1292 - build(deps): bump github.com/cristalhq/acmd from 0.8.1 to 0.11.0

Pull Request - State: closed - Opened by dependabot[bot] about 2 years ago - 1 comment
Labels: dependencies, go

#1290 - checkers/wrapperFunc: suggest `strings.Contains*` instead of `strings.Index* >= 0`

Pull Request - State: closed - Opened by tmzane about 2 years ago - 1 comment

#1289 - checkers/wrapperFunc: suggest `strings.Cut` instead of `strings.Index`

Pull Request - State: closed - Opened by tmzane about 2 years ago - 3 comments

#1288 - caseOrder: type is not defined if type is defined in another file in the same package

Issue - State: open - Opened by Ivan-Feofanov about 2 years ago - 3 comments
Labels: question

#1287 - checkers: expand supported conditions for badCondition checker

Pull Request - State: closed - Opened by peakle over 2 years ago - 1 comment

#1286 - Generic default value generates newDeref error

Issue - State: closed - Opened by Bechma over 2 years ago - 5 comments
Labels: bug

#1281 - checkers: removed timeCmpSimplify checker

Pull Request - State: open - Opened by quasilyte over 2 years ago - 1 comment

#1276 - ruleguard: load rules once

Pull Request - State: open - Opened by peakle over 2 years ago - 3 comments

#1271 - docs: update

Pull Request - State: closed - Opened by cristaloleg over 2 years ago - 1 comment

#1270 - docs/overview.md: dynamicFmtString missing from checkers documentation.

Issue - State: closed - Opened by ErgoAsciiCat over 2 years ago - 2 comments
Labels: checker stabilization, ruleguard

#1267 - `unparam` linter cannot handle parameters with generic placeholders

Issue - State: closed - Opened by abdusco over 2 years ago - 6 comments

#1257 - timeCmpSimplify rule is wrong

Issue - State: open - Opened by ferhatelmas over 2 years ago - 6 comments
Labels: checker stabilization

#1244 - commentedOutCode does not detect commented out assignments

Issue - State: open - Opened by rittneje over 2 years ago - 3 comments
Labels: good first issue, checker stabilization

#1241 - lintmain/internal: panic on loading a package ending with _test

Issue - State: closed - Opened by reillywatson over 2 years ago - 1 comment
Labels: bug

#1226 - nolint comment is ignored

Issue - State: closed - Opened by mpldr over 2 years ago - 4 comments
Labels: question

#1216 - condition not found

Issue - State: closed - Opened by maddin200 almost 3 years ago - 4 comments

#1212 - checkers: remove the `ioutil` check

Pull Request - State: closed - Opened by jeffwidman almost 3 years ago - 12 comments

#1200 - hugeParam-ref and rangeValCopy default limit is extremely low

Issue - State: closed - Opened by Bysmyyr about 3 years ago - 6 comments
Labels: discussion

#1196 - checkers/missingErrorAccess_checker: new checker

Pull Request - State: open - Opened by komkom about 3 years ago

#1183 - Investigate the performance regression reason

Issue - State: open - Opened by quasilyte about 3 years ago
Labels: bug

#1177 - Move code related to embedded rules from checkers/checkers.go to checkers/embedded_rules.go

Issue - State: closed - Opened by quasilyte about 3 years ago - 1 comment
Labels: help wanted, good first issue

#1170 - commentedOutCode: false negative if preceded by prose

Issue - State: open - Opened by rittneje about 3 years ago - 1 comment
Labels: question

#1168 - Consider ignoring some common methods in hugeParam checker

Issue - State: open - Opened by quasilyte about 3 years ago - 1 comment
Labels: enhancement, help wanted, good first issue

#1157 - Panic after upgrade golangci-lint to v1.43.0

Issue - State: closed - Opened by Antonboom about 3 years ago - 15 comments

#1154 - checkers: extend `preferFprint` for `WriteString` calls

Pull Request - State: closed - Opened by tdakkota about 3 years ago - 3 comments

#1152 - gocritic: ruleguard init error could not import github.com/quasilyte/go-ruleguard/dsl

Issue - State: open - Opened by sebastien-rosset about 3 years ago - 1 comment
Labels: bug, ruleguard

#1148 - Find cutsets of strings.{Trim,TrimLeft,TrimRight}, which are not in order

Issue - State: open - Opened by breml over 3 years ago - 2 comments
Labels: checker request

#1135 - go-toolsmith/[email protected] panics in some modules

Issue - State: closed - Opened by MichaelUrman over 3 years ago - 8 comments
Labels: bug

#1126 - panic in load embedded ruleguard rules: can't load fmt

Issue - State: closed - Opened by egonelbre over 3 years ago - 44 comments

#1108 - checkers: add unassignedError

Pull Request - State: open - Opened by cristaloleg over 3 years ago - 4 comments

#1090 - Fix release drafts

Issue - State: closed - Opened by quasilyte over 3 years ago - 1 comment

#1081 - dupSubExpr: rewrite with ruleguard

Pull Request - State: open - Opened by cristaloleg over 3 years ago

#1068 - feat: add new rule check for passing large structs to func receivers

Issue - State: open - Opened by geraldywy over 3 years ago - 1 comment
Labels: enhancement

#1037 - Add -enable and -disable flags for the ruleguard checker

Issue - State: open - Opened by quasilyte almost 4 years ago
Labels: enhancement, good first issue

#1022 - exitAfterDefer and explicitly cancel()

Issue - State: closed - Opened by Antonboom almost 4 years ago - 8 comments

#1016 - Automate docs publication

Issue - State: open - Opened by cristaloleg about 4 years ago
Labels: enhancement

#1004 - Missed documentation for ruleguard check

Issue - State: closed - Opened by orsinium about 4 years ago - 1 comment

#1003 - Giving caseOrder error erroneously

Issue - State: closed - Opened by echarrod about 4 years ago - 3 comments
Labels: bug

#993 - dev: added makefile step to generate new checker

Pull Request - State: closed - Opened by neverix over 4 years ago - 2 comments

#985 - Enable more go-critic checks that we run on go-critic

Issue - State: open - Opened by quasilyte over 4 years ago

#982 - Add one or more Go projects to the integration tests suite

Issue - State: open - Opened by quasilyte over 4 years ago - 7 comments

#976 - dev: makefile step to init new checker

Issue - State: closed - Opened by cristaloleg over 4 years ago
Labels: help wanted, good first issue, Hacktoberfest

#944 - Extend golangci-lint documentation to describe go-critic configuration in more details

Issue - State: open - Opened by quasilyte over 4 years ago - 3 comments
Labels: discussion

#863 - yodaStyleExpr should support in-range operations

Issue - State: open - Opened by korya over 5 years ago - 1 comment
Labels: bug

#699 - lint/internal/astwalk: localDefWalker should visit short stmt in for/switch/etc

Issue - State: open - Opened by quasilyte over 6 years ago - 1 comment
Labels: bug

#645 - lint: multi-value case causes emptyFallthrough false positives

Issue - State: open - Opened by quasilyte over 6 years ago - 1 comment
Labels: bug

#614 - lint: remove go-tools/cmd/unused linter workarounds for Context.require field

Issue - State: closed - Opened by quasilyte over 6 years ago - 1 comment

#566 - Question regarding removing checks

Issue - State: closed - Opened by iggyzap over 6 years ago - 4 comments