Skip to content
This repository has been archived by the owner on Jan 13, 2025. It is now read-only.

add coverage to coveralls.io #21

Merged
merged 1 commit into from
Jan 10, 2025
Merged

add coverage to coveralls.io #21

merged 1 commit into from
Jan 10, 2025

Conversation

jhheider
Copy link
Contributor

we reference it int the readme (for pkgxdev/pkgx) so we might as well have it ready for when we merge. though the number will be bad (no significant coverage).

@mxcl
Copy link
Member

mxcl commented Jan 10, 2025

I intend to have copilot write them.

@jhheider jhheider force-pushed the coverage branch 3 times, most recently from 820b3c5 to b6d0d01 Compare January 10, 2025 19:57
@jhheider
Copy link
Contributor Author

jhheider commented Jan 10, 2025

Coverage Status. gotta start somewhere.

@jhheider jhheider marked this pull request as ready for review January 10, 2025 20:03
@jhheider jhheider requested a review from mxcl January 10, 2025 20:03
@mxcl
Copy link
Member

mxcl commented Jan 10, 2025

boast worthy in its lowness

@jhheider
Copy link
Contributor Author

we're proud of our untested code. see how damn good it is.

we reference it int the readme (for `pkgxdev/pkgx`) so we might as well have it ready for when we merge. though the number will be bad (no significant coverage).
@mxcl mxcl merged commit 0340057 into main Jan 10, 2025
13 checks passed
@mxcl mxcl deleted the coverage branch January 10, 2025 20:18
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants