-
-
Notifications
You must be signed in to change notification settings - Fork 11.9k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
chore: track code coverage in PRs #5312
base: main
Are you sure you want to change the base?
Conversation
This comment was marked as resolved.
This comment was marked as resolved.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this is okay, but @avelino needs to approve.
This comment was marked as resolved.
This comment was marked as resolved.
general review: I like the idea of putting this action |
.github/workflows/tests.yaml
Outdated
run: go get -v -t -d ./... | ||
- name: Run tests | ||
run: go test main_test.go main.go | ||
if: steps.verify-changed-files.outputs.files_changed == 'true' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We need to run tests still, only skip the coverage step please.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
it looks like it's not working either way :(
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@kilianc can you enable github actions and github pages for your fork for cover branch and try and run it.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done: kilianc#1 (comment)
but the action to detect changes is borked, I am going to use git directly or find a new one if that's ok
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sure
It would be nice to track code coverage for this repo
Summary by CodeRabbit