Improvements

• extend flagName diagnostics @quasilyte (#1005)
• enable ruleguard debug via [email protected]=<name> @quasilyte (#1002)
• ruleguard now supports multi rules input @quasilyte (#1001)
• improve exitAfterDefer warnings text @josephzidell (#978)

New checkers

• added badLock checker @quasilyte (#999)
• added builtinShadowDecl @quasilyte (#970)
• added typeDefFirst checker @LevOspennikov (#960)

Fixes

• fix stringXbytes @cristaloleg (#1013)
• fix unlambda false positive on ptr vars @quasilyte (#1008)
• fix unlambda panic on variadic func literals @quasilyte (#992)
• ignore defer exit() in exitAfterDefer @quasilyte (#996)
• don't warn on nolint with reason in commentFormatting @quasilyte (#986)
• make argOrder checker more conservative @quasilyte (#969)

Changes

• checkers: mark more checkers as stable @quasilyte (#956)

Checkers that are now considered to be stable:

* argOrder
* badCall
* badCond
* codegenComment
* commentFormatting
* deprecatedComment
* exitAfterDefer
* flagName
* mapKey
* newDeref
* offBy1
* stringXbytes
* valSwap
* wrapperFunc

Every checker from this list was tested on several big Go projects.

Updated go-ruleguard version to v0.2.0.

This minor release is an attempt to fix a broken dependency.
See #953

It's now possible to install go-critic with go get (no lintpack required).

New checkers:

• 9a11c85 added deferUnlambda checker.
• 99489e1 added badRegexp checker.
• 3a3e885 added regexpSimplify checker.
• 3916229 added sloppyTypeAssert checker.
• d6fa699 added sqlQuery checker.
• 33d100b added sortSlice checker.

Fixes:

• 4757c17 typeSwitchVar now generates 1 warning per switch statement.
• 7668b86 commentFormatting does not report //go:generate comments.
• 730db5a commentFormatting does not report //export comments.
• f4eb84b unnecessaryDefer false positive fix.

Changes

• checkers: add ruleguard checker @quasilyte (#907)
• add draw.Draw to draw.DrawMask checker @seregayoga (#904)
• checkers: rename deferAtTheEnd to unnecessaryDefer @quasilyte (#902)
• checker: add defer at the end of function checker @janisz (#901)

Changes

• all: rename release-config.yaml to release-drafter.yml @cristaloleg (#898)
• readme: add "cd" step to installation guide @quasilyte (#896)
• checkers: add mapKey checker @quasilyte (#895)
• checkers: fix integration tests @quasilyte (#894)
• all: use Github Actions @cristaloleg (#891)
• all: make automate releases @cristaloleg (#892)
• makefile: correct linter.version argument @sosiska (#890)

Long time ago we did v0.3.4 release, it's time to make them more regular.
Current one contains mostly bugfixes.

• Fixed a number of false positives and panics for stable checkers
• Moved to a lintpack framework which uses https://godoc.org/golang.org/x/tools/go/packages
• Improved the test suite for checkers
• Moved less maintained and too experimental checkers to contrib repo

Marked several checkers stable.
The current list of stable checkers:

appendAssign [diagnostic]
appendCombine [performance]
assignOp [style]
builtinShadow [style opinionated]
captLocal [style]
caseOrder [diagnostic]
defaultCaseOrder [style]
dupArg [diagnostic]
dupBranchBody [diagnostic]
dupCase [diagnostic]
dupSubExpr [diagnostic]
elseif [style]
flagDeref [diagnostic]
hugeParam [performance]
ifElseChain [style]
importShadow [style opinionated]
indexAlloc [performance]
paramTypeCombine [style opinionated]
rangeExprCopy [performance]
rangeValCopy [performance]
regexpMust [style]
singleCaseSwitch [style]
sloppyLen [style]
switchTrue [style]
typeSwitchVar [style]
typeUnparen [style opinionated]
underef [style]
unlambda [style]
unslice [style]