Skip to content
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

publish coverage in github pull requests #33

Merged
merged 1 commit into from
Mar 20, 2025
Merged

publish coverage in github pull requests #33

merged 1 commit into from
Mar 20, 2025

Conversation

cyrilst
Copy link
Contributor

@cyrilst cyrilst commented Mar 11, 2025

github-actions will automatically post a comment in the PR with the coverage report. On subsequent pushes in the PR, the comment will be updated. Once all files have a good enough coverage, we will raise the minimum allowed coverage.

@cyrilst cyrilst self-assigned this Mar 11, 2025
@cyrilst cyrilst force-pushed the coverage branch 4 times, most recently from 3bdc9f3 to 569f83a Compare March 11, 2025 16:04
Copy link

github-actions bot commented Mar 11, 2025

File Coverage
All files 2%
cmd/sshproxy-dumpd/sshproxy-dumpd.go 0%
cmd/sshproxy-replay/sshproxy-replay.go 0%
cmd/sshproxy/commands.go 0%
cmd/sshproxy/recorder.go 0%
cmd/sshproxy/sshproxy.go 0%
cmd/sshproxyctl/sshproxyctl.go 0%
pkg/nodesets/nodesets.go 0%
pkg/record/record.go 0%
pkg/utils/config.go 0%
pkg/utils/duration.go 0%
pkg/utils/etcd.go 0%
pkg/utils/logging.go 0%
pkg/utils/route.go 0%
pkg/utils/utils.go 92%

Minimum allowed coverage is 0%

Generated by 🐒 cobertura-action against ea71329

@cyrilst cyrilst force-pushed the coverage branch 4 times, most recently from a1aaa2f to 3144da9 Compare March 11, 2025 16:45
@cea-hpc cea-hpc deleted a comment from github-actions bot Mar 11, 2025
@cyrilst cyrilst marked this pull request as ready for review March 11, 2025 19:17
@cyrilst cyrilst requested review from t-valette, fdiakh and od-cea March 11, 2025 19:17
Copy link
Member

@od-cea od-cea left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Excellent !

@cyrilst cyrilst merged commit 659cb94 into master Mar 20, 2025
8 checks passed
@cyrilst cyrilst deleted the coverage branch March 20, 2025 15:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants