polyfloyd / go-errorlint

A source code linter that can be used to find code that will cause problems with Go's error wrapping scheme
MIT License
248 stars 19 forks source link

ci: do run Go tests #53

Closed kolyshkin closed 1 year ago

kolyshkin commented 1 year ago

Apparently, what happens in GHA CI now is:

go test -test.v -race -cover shell: /usr/bin/bash -e {0} ? github.com/polyfloyd/go-errorlint [no test files]

Fix this.

Note that this is why we missed an issue with #47 (the fix is in #52). Without the fix in #52 merged, this PR should make CI fail.

kolyshkin commented 1 year ago

Yes, this works as expected (CI is "red" because of an issue fixed by #52).

@polyfloyd I think it makes sense to merge this one first (so that we're able to actually run CI on other PRs).

polyfloyd commented 1 year ago

Thanks!