mvdan/unparam issues and pull requests

#74 - Allow enable on constructors like `NewXXX`

Issue - State: open - Opened by pengubco 9 months ago

#73 - Bump golang.org/x/tools to v0.7.0 (as minimum) and make new tag

Issue - State: closed - Opened by Antonboom over 1 year ago - 2 comments

#72 - fix more crashes with type params

Pull Request - State: closed - Opened by mgabeler-lee-6rs over 1 year ago - 4 comments

#71 - panic analysing generic function with struct param with generic field types

Issue - State: closed - Opened by duncanharris over 1 year ago - 5 comments

#70 - update golang.org/x/tools, fix panic with type params

Pull Request - State: closed - Opened by mgabeler-lee-6rs over 1 year ago - 1 comment

#69 - Add option to configure min number of call sites

Pull Request - State: closed - Opened by duncanharris over 1 year ago - 2 comments

#68 - ignore underscore-prefixed parameters

Pull Request - State: closed - Opened by favonia over 1 year ago

#67 - An option to ignore parameters starting with the underscore (e.g., `_name`)?

Issue - State: closed - Opened by favonia over 1 year ago - 2 comments

#66 - disable warnings on funcs which use go:linkname

Issue - State: open - Opened by mvdan almost 2 years ago

#65 - add Go 1.19.x, drop 1.17.x

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

#64 - add preliminary support for generic code

Pull Request - State: closed - Opened by mvdan about 2 years ago - 4 comments

#63 - fix: panic related to IndexListExpr

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

#62 - panic: interface conversion: ast.Expr is ast.IndexExpr, not ast.Ident

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

#60 - panics when encountering generics

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

#59 - fails on itself at tip

Issue - State: closed - Opened by jba over 2 years ago - 1 comment
Labels: good first issue

#58 - proposal: don't group exported and unexported methods of an interface together?

Issue - State: closed - Opened by ainar-g over 2 years ago

#57 - go.mod: update golang.org/x/tools

Pull Request - State: closed - Opened by ainar-g almost 3 years ago

#56 - go.mod: update golang.org/x/tools

Pull Request - State: closed - Opened by ainar-g almost 3 years ago

#55 - Support Go 1.17 slice-to-array conversion

Issue - State: closed - Opened by ainar-g about 3 years ago - 5 comments

#54 - Add support for setting tests field on Checker type

Pull Request - State: closed - Opened by bfbachmann about 3 years ago - 1 comment

#53 - Allow setting of tests struct field on the Checker type

Issue - State: closed - Opened by bfbachmann about 3 years ago - 1 comment

#52 - False positive on implementing interface

Issue - State: open - Opened by robinknaapen about 3 years ago - 4 comments

#51 - github: update go versions

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

#50 - github: update to actions/setup-go@v2 and go versions

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

#49 - false negative

Issue - State: closed - Opened by helmus about 4 years ago - 5 comments

#48 - Possible to skip directories through flag?

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

#47 - Upgrade golang.org/x/tools to v0.0.0-20200426102838-f3a5411a4c3b

Pull Request - State: closed - Opened by atombender about 4 years ago

#46 - main package has no godoc

Issue - State: closed - Opened by dmitshur over 4 years ago - 1 comment

#45 - mvdan.cc certificate expired

Issue - State: closed - Opened by piotrbulinski over 4 years ago - 3 comments

#44 - goreleaser: error: unknown long flag '--snapshot', try --help

Issue - State: closed - Opened by dzpao over 4 years ago - 2 comments

#43 - How to suppress the error in this situation?

Issue - State: closed - Opened by dzpao over 4 years ago - 6 comments
Labels: needs info

#42 - Ignoring leading underscore _parameters

Issue - State: closed - Opened by jpalardy over 4 years ago - 9 comments

#41 - Is this positive case?

Issue - State: closed - Opened by hollowaykeanho over 4 years ago - 1 comment

#40 - False positive about result err is always nil

Issue - State: open - Opened by lsytj0413 almost 5 years ago - 6 comments

#39 - Possible to ignore stubbed functions?

Issue - State: closed - Opened by ansermino about 5 years ago - 4 comments
Labels: needs info

#38 - Go-RPC

Issue - State: closed - Opened by 1995parham over 5 years ago - 3 comments

#37 - False positive about unused return values

Issue - State: closed - Opened by cloudlena over 5 years ago - 5 comments

#36 - Flags necessary elements of function signature

Issue - State: closed - Opened by akerl over 5 years ago - 3 comments

#35 - Issue on function that have parameters that shouldn't be removed

Issue - State: closed - Opened by mavimo over 5 years ago - 2 comments

#34 - set exit code according to found warnings

Pull Request - State: closed - Opened by sni over 5 years ago - 6 comments

#33 - Use github repo address instead of mvdan.cc

Pull Request - State: closed - Opened by chilladx over 5 years ago - 1 comment

#32 - Assigning an unused parameter to a blank identifier doesn't make it used.

Issue - State: closed - Opened by dmitshur over 5 years ago - 2 comments

#31 - Remove the "always receives X" check entirely

Issue - State: open - Opened by mvdan almost 6 years ago - 12 comments
Labels: help wanted

#30 - Fix crash on nil constant

Pull Request - State: closed - Opened by jirfag about 6 years ago - 2 comments

#29 - Add algo and exported fields setters for Checker

Pull Request - State: closed - Opened by jirfag about 6 years ago

#28 - check: use "comma,ok" for all type assertions

Pull Request - State: closed - Opened by quasilyte about 6 years ago - 2 comments

#27 - Detect when args are always redundant because of another argument

Issue - State: open - Opened by mvdan over 6 years ago

#26 - check: don't consider blank identifier as descriptive name

Pull Request - State: closed - Opened by dmitshur over 6 years ago - 2 comments

#25 - Don't consider the blank identifier as having a name.

Issue - State: closed - Opened by dmitshur over 6 years ago - 5 comments

#24 - don't warn when removing all result parameters would add complexity to the code

Issue - State: open - Opened by mvdan over 6 years ago - 1 comment

#23 - false positive with func called via reflect.ValueOf(x).Call

Issue - State: closed - Opened by dvic over 6 years ago - 13 comments

#22 - unparam does not support build tags

Issue - State: closed - Opened by benpaxton-hf over 6 years ago - 7 comments

#21 - Potential false negative, didn't report error result in unexported method that always returned nil.

Issue - State: closed - Opened by dmitshur over 6 years ago - 12 comments

#20 - Flag to ignore unused error return parameters

Issue - State: closed - Opened by tehsphinx over 6 years ago - 8 comments

#19 - Unnamed parameters and test doubles

Issue - State: closed - Opened by gonzaloserrano almost 7 years ago - 3 comments

#18 - Detect return parameters that are just a receiving parameter

Issue - State: closed - Opened by mvdan almost 7 years ago - 7 comments
Labels: enhancement

#17 - Detect return parameters that always have the same value

Issue - State: closed - Opened by mvdan almost 7 years ago - 3 comments
Labels: enhancement

#16 - detect cases where no caller uses a return value

Issue - State: closed - Opened by kevinburke almost 7 years ago - 2 comments
Labels: enhancement

#15 - "X always receives Y" false positive with runtime constant

Issue - State: closed - Opened by mvdan almost 7 years ago - 1 comment
Labels: bug

#14 - Don't say "always receives <value>" when the value is passed exactly once

Issue - State: closed - Opened by mvdan almost 7 years ago - 1 comment
Labels: help wanted

#13 - Potential false positive or confusing/inconsistent report.

Issue - State: closed - Opened by dmitshur almost 7 years ago - 8 comments

#12 - unparam -help doesn't print usage.

Issue - State: closed - Opened by dmitshur almost 7 years ago - 4 comments

#11 - Detect unexported func parameters that always receive the same value

Issue - State: closed - Opened by mvdan about 7 years ago - 3 comments
Labels: enhancement

#10 - Panic on Go 1.8 struct conversion ignoring field tags.

Issue - State: closed - Opened by dmitshur about 7 years ago - 1 comment

#9 - Detect unused params used in recursive calls

Issue - State: closed - Opened by mvdan about 7 years ago - 1 comment
Labels: enhancement

#8 - Also flag dead arguments

Issue - State: closed - Opened by mvdan about 7 years ago
Labels: enhancement

#7 - Be smarter about what func signatures to ignore

Issue - State: closed - Opened by mvdan about 7 years ago - 3 comments
Labels: enhancement

#6 - unparam doesn't detect this offender

Issue - State: closed - Opened by metaleap about 7 years ago - 9 comments

#5 - Common linter interface

Issue - State: closed - Opened by mvdan over 7 years ago - 8 comments
Labels: help wanted

GitHub / mvdan/unparam issues and pull requests