-
Notifications
You must be signed in to change notification settings - Fork 397
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(ci): add codecov #4064
chore(ci): add codecov #4064
Conversation
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.
@rahul2393 I've put some thoughts. 👍🏼
- name: Upload coverage reports to Codecov | ||
uses: codecov/[email protected] | ||
with: | ||
token: ${{ secrets.CODECOV_TOKEN }} |
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.
Is this secret already set?
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.
@yanivagman to confirm.
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.
Changes are already tested on fork repo https://github.com/rahul2393/tracee/pull/6
.github/workflows/pr.yaml
Outdated
@@ -224,6 +224,11 @@ jobs: | |||
- name: Run Unit Tests | |||
run: | | |||
make test-unit | |||
- name: Upload coverage reports to Codecov | |||
uses: codecov/[email protected] |
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.
Would you mind using this pattern?
...@hash # vX.X.X
tracee/.github/workflows/pr.yaml
Line 128 in 00598de
uses: actions/checkout@b4ffde65f46336ab88eb53be808477a3936bae11 # v4.1.1 |
@@ -224,6 +224,11 @@ jobs: | |||
- name: Run Unit Tests | |||
run: | | |||
make test-unit | |||
- name: Upload coverage reports to Codecov | |||
uses: codecov/codecov-action@e0b68c6749509c5f83f984dd99a76a1c1a231044 #v4.0.1 |
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.
Nice!
The latest version is v4.4.1. Is there a reason for not using it? If so, please insert a comment explaining it.
1. Explain what the PR does
Fixes: #152
2. Explain how to test it
3. Other comments