lostluck commented on code in PR #26087:
URL: https://github.com/apache/beam/pull/26087#discussion_r1157655609
##########
.github/workflows/go_tests.yml:
##########
@@ -48,7 +48,7 @@ jobs:
- name: Delete old coverage
run: "cd sdks/go/pkg && rm -rf .coverage || :"
- name: Run coverage
- run: cd sdks/go/pkg && go test -coverprofile=coverage.txt
-covermode=atomic ./...
+ run: cd sdks && go test -coverprofile=coverage.txt -covermode=atomic
./go/pkg/... ./go/container/... ./java/container/... ./python/container/...
./typescript/container/...
Review Comment:
Enabling covbot here would be out of scope for this change I think. The go
tests is different because it's otherwise hard to ensure an action will run,
without having a change that ensures the action runs and that was fixing a
clear gap.
--
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]
