Skip to content

Commit 5e4b700

Browse files
redsun82Copilot
andcommitted
Go: de-bazel checks in test action
The codegen drift and gofmt checks move to `semmle-code`, so drop them here to avoid duplication and replace the bazel build with a plain `go build ./...` sanity check. Keep the qhelp-to-markdown steps. Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
1 parent 05c40ea commit 5e4b700

1 file changed

Lines changed: 4 additions & 21 deletions

File tree

go/actions/test/action.yml

Lines changed: 4 additions & 21 deletions
Original file line numberDiff line numberDiff line change
@@ -1,12 +1,12 @@
11
name: Test Go extractor
2-
description: Run build, QL tests, and optionally basic code sanity checks (formatting and generated code) for the Go extractor
2+
description: Run build, QL tests, and optionally qhelp generation for the Go extractor
33
inputs:
44
go-test-version:
55
description: Which Go version to use for running the tests
66
required: false
77
default: "~1.26.6"
88
run-code-checks:
9-
description: Whether to run formatting, code and qhelp generation checks
9+
description: Whether to run qhelp generation checks
1010
required: false
1111
default: false
1212
runs:
@@ -26,28 +26,11 @@ runs:
2626
shell: bash
2727
run: 'find .github/problem-matchers -name \*.json -exec echo "::add-matcher::{}" \;'
2828

29-
- name: Check checked-in generated code
30-
if: inputs.run-code-checks == 'true'
31-
shell: bash
32-
run: |
33-
bazel run go:gen
34-
git add .
35-
git diff --exit-code HEAD || (
36-
echo "please run bazel run //go:gen"
37-
exit 1
38-
)
39-
4029
- name: Build
4130
shell: bash
4231
run: |
43-
bazel run go:go-installer
44-
45-
- name: Check that all Go code is autoformatted
46-
if: inputs.run-code-checks == 'true' && !cancelled()
47-
shell: bash
48-
run: |
49-
cd go
50-
make check-formatting
32+
cd go/extractor
33+
go build ./...
5134
5235
- name: Compile qhelp files to markdown
5336
if: inputs.run-code-checks == 'true' && !cancelled()

0 commit comments

Comments
 (0)