Add a lint action

[lestrrat]

fixes #105

[lestrrat]

@goccy というわけで当然テストは落ちます！ :)

[goccy]

timeout しちゃってるみたいなので、何かしら設定してあげる必要がありそうです〜！

[lestrrat]

これ、ローカルでもたまにあるんですけど、何回か走らせてると普通に動くんですよね…

[lestrrat]

--timeou=5m ででるようになったみたいです。

これはちなみに、ですけど、go-jsonは他のプロジェクトに比べて明らかにgolangci-lintが遅い気がします！

[goccy]

自動生成ファイルとか、一部のカバレッジ向上目的のでかいテストファイルを除いたりすると多少改善するかな...
といった感じですかねえ。そのあたりは自分の方で調整してみようと思います。

[goccy]

一度 master をとりこんでいただくと、 lint の error が解消されるかと思います...！
( .golangci.yml がコンフリクトしてしまったので、 disable linter のまわりだけちょっといじらないといけないかもです )

[lestrrat]

僕の手元ではまだちょっとだけ残ってるみたいです。最初のやつはなんかそこだけ無視指定でいい気もしますが、最後の三つは対処したほうがよさそう。ちなみにgolangci-lint、すごいペースで新しいバージョンが出るので、僕と @goccy さんのバージョンがあってないとかもありそう。

ちなみに今見たらActionsに指定してたバージョンも古かったので一応このあと最新版にひきあげておきます(1.36.0)

lestrrat@finch go-json % golangci-lint run ./...
encode_string.go:400:37: unsafeptr: possible misuse of reflect.SliceHeader (govet)
	return *(*[]uint64)(unsafe.Pointer(&reflect.SliceHeader{
	                                   ^
decode_slice.go:33:5: variable cap has same name as predeclared identifier (predeclared)
				cap := 2
				^
decode_slice.go:87:4: variable cap has same name as predeclared identifier (predeclared)
			cap := slice.cap
			^
decode_slice.go:187:4: variable cap has same name as predeclared identifier (predeclared)
			cap := slice.cap
			^

[goccy]

v1.36.0 で通ったものを master に入れておきました。

go vet ./... で出る以下のエラーは false positive なので ( 大体 encoding/json に存在するテストケース起因のもの )、
とりあえず golangci-lint だけでも良さそうでした。

# github.com/goccy/go-json_test
./decode_test.go:2363:2: struct field m has json tag but is not exported
./decode_test.go:2364:2: struct field m2 has json tag but is not exported
./decode_test.go:2366:2: struct field s has json tag but is not exported
./encode_test.go:107:4: struct field a has json tag but is not exported
./encode_test.go:440:4: struct field a has json tag but is not exported
./tagkey_test.go:62:2: struct field tag `:"BadFormat"` not compatible with reflect.StructTag.Get: bad syntax for struct tag key

[goccy]

検証のために #114 を作成してみました。

[lestrrat]

ああ。go vetととればいいのかな
まぁ僕の場合それで探せてなかったケースがいくつかあった、ってだけなので…