kpumuk commented on code in PR #3493:
URL: https://github.com/apache/thrift/pull/3493#discussion_r3266894003
##########
.github/workflows/sca.yml:
##########
@@ -202,12 +202,53 @@ jobs:
with:
go-version: ${{ matrix.go }}
- - name: Run gofmt
+ - name: Install dependencies
+ run: |
+ sudo apt-get update -yq
+ # shellcheck disable=SC2086
+ sudo apt-get install -y --no-install-recommends g++ $BUILD_DEPS
+
+ - name: Configure
+ run: |
+ ./bootstrap.sh
+ # shellcheck disable=SC2086
+ ./configure $(echo $CONFIG_ARGS_FOR_SCA | sed
's/without-go/with-go/')
+
+ - uses:
actions/download-artifact@3e5f45b2cfb9172054b4087a40e8e0b5a5461e7c # v8.0.1
+ with:
+ name: thrift-compiler
+ path: compiler/cpp
+
+ - name: Run gofmt on checked-in Go files
# Args:
# -s: Simplify code
- # -d: Display changes, and exit non-zero when changes present
+ # -l: List files that would change
# -e: Report all errors instead of only the first 10
- run: gofmt -s -d -e $(git ls-files | grep "\.go$")
+ run: |
+ git ls-files '*.go' > /tmp/go-files.txt
+ xargs -r gofmt -s -l -e < /tmp/go-files.txt > /tmp/gofmt-files.txt
Review Comment:
Ha, an example just came through
https://github.com/apache/thrift/actions/runs/26098116881/job/76747159994?pr=3495
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
