Repository navigation
Add CodeQL analysis for Go code - #957
Conversation
|
You are seeing this message because GitHub Code Scanning has recently been set up for this repository, or this pull request contains the workflow file for the Code Scanning tool. What Enabling Code Scanning Means:
For more information about GitHub Code Scanning, check out the documentation. |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## develop #957 +/- ##
===========================================
- Coverage 63.94% 63.94% -0.01%
===========================================
Files 758 758
Lines 53249 53241 -8
Branches 8446 8443 -3
===========================================
- Hits 34051 34045 -6
+ Misses 17535 17533 -2
Partials 1663 1663 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
There was a problem hiding this comment.
🟢 Approval recommended
The workflow matches the Go CGO build requirements and the referenced four-language CodeQL run completed successfully.
Pull request overview
Adds CodeQL analysis for the CGO-based Go module while preserving existing language scans.
Changes:
- Adds a manual Go CodeQL matrix job.
- Builds native TsFile dependencies before traced Go compilation.
- Renames the Java matrix identifier while retaining Java-only analysis.
File summaries
| File | Description |
|---|---|
.github/workflows/codeql.yml |
Configures Go prerequisites, native build, and CodeQL-traced compilation. |
Review details
- Files reviewed: 1/1 changed files
- Comments generated: 0
- Review effort level: Balanced
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
Add CodeQL analysis for the Go module using
build-mode: manual. The Go API uses CGO, so analysis needs the native TsFile headers and library; without them, extraction reports a missingcwrapper/errno_define_c.hheader and cannot resolveimport "C".The Go job sets up Java and Go, installs
uuid-dev, and builds the native library before initializing CodeQL. It then runsCGO_ENABLED=1 go build -a ./...fromgo/under CodeQL tracing. The-aflag rebuilds cached packages so their compilation is captured. The existing manual C++ build step is restricted to the C++ matrix entry.The Java matrix entry is also renamed from
java-kotlintojava, retainingbuild-mode: none.Validation:
actionlint, native library build, andCGO_ENABLED=1 go build -a ./...passed./language:java, whiledevelopuses/language:java-kotlin.