Martin Tournoij

[Their tests](https://github.com/matomo-org/device-detector/blob/master/Tests/fixtures/bots.yml); looks looks like they just use this [list of regexps](https://github.com/matomo-org/device-detector/blob/master/regexes/bots.yml) based on [Bot.php](https://github.com/matomo-org/device-detector/blob/master/Parser/Bot.php).

SA9009: flag defer foo() where foo returns a closure

2

Add a new checker to flag cases where a deferred function returns exactly one function, and that function isn't called. For example: func f() func() { // Do stuff. return...

SA1032: add check for errors.Is() and .As() argument order

1

Fixes #1530

lintcmd: don't print \' in -list-checks and -explain

1

All the single quotes are written as `\'` in the descriptions. I assume that's for a good reason, but -list-checks would print: % staticcheck -list-checks | grep "'" S1003 Replace...

Flag all stdlib functions that have both a Printf() and Print() variant. Found with: % rg -g '!internal/' -g '!cmd/' '^func ([()*a-zA-Z0-9_ ]+)?[A-Z]\w+\(\w+ string, \w+ \.\.\.any\)' (will need to filter...

This generally makes code less clear; e.g. the "fixed" version for the added test would be: if strings.HasPrefix(data, "\xff\xfe") { data = data[2:] } else { data = strings.TrimPrefix("\xef\xbb\xbf") }...

structlayout: align output

2

This aligns the output of structlayout. To my eyes at least, this is a lot more readable, including more readable than the structlayout-pretty output. Before: % structlayout bufio Reader column.name...

Only skip if both branches are documented; in cases where just one is commented it can just be: // Comment it. return [..] Closes: gh-704 Closes: gh-1488

SA4006: Fix for compound operators, increment/decrement

1

In the case of compound operators such as +=, -=, etc we need to use the left-hand side to get the value. For IncDec we need a different codepath from...

Fixes #793