Go: Add language-specific baseline configuration #13846
Conversation
|
@smowton Note that CI failed because it found Go 1.20.5 in the cache and used that |
The toolcache version used in an Actions run on a repo isn't monotonic, and can wobble during the rollout of a new version of the toolcache. So it might be an idea to set up a specific patch version of Go, if the tests rely on that. |
|
@henrymercer Sure. Done (though the new commit that I've pushed isn't showing up yet - maybe a short delay?). Note that I wasn't sure whether to put this in |
|
Looking at the Makefile, |
|
I looked at your branch and it looks good, except I think |
fe9c8dd
to
47a536c
Compare
There was a problem hiding this comment.
I built an intree dist for Go and tested this on a repo with vendored modules. After making the modifications below, it worked 🎉:
> codeql database init path/to/db --source-root path/to/source --language go -vvv --calculate-language-specific-baseline
...
Ignored an additional 2516 files when processing baseline information for Go due to paths and paths-ignore configuration.
Found 1254 baseline files for go.
...
Co-authored-by: Henry Mercer <henry.mercer@me.com>
No description provided.