Skip to content
This repository has been archived by the owner on Aug 9, 2023. It is now read-only.

ci: add tests and coverage #15

Merged
merged 1 commit into from
Jul 21, 2023
Merged

ci: add tests and coverage #15

merged 1 commit into from
Jul 21, 2023

Conversation

aasseman
Copy link
Collaborator

@aasseman aasseman commented Jul 20, 2023 •
edited
Loading

Added:

  • test + coverage using cargo llvm-cov test
  • Added cargo test for the documentation code snippets
  • Added coverage reporting using coveralls (there isn't even a need to create an account or anything!)

Fixes #4

@aasseman aasseman requested a review from hopeyen July 20, 2023 18:40
@aasseman aasseman self-assigned this Jul 20, 2023
@aasseman
ci: add tests and coverage
c24d40c 
Fixes #4

Signed-off-by: Alexis Asseman <[email protected]>
@aasseman aasseman marked this pull request as ready for review July 21, 2023 00:28
hopeyen
hopeyen approved these changes Jul 21, 2023
View reviewed changes
Copy link
Owner

@hopeyen hopeyen left a comment

Choose a reason for hiding this comment

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

cool:D! Feel free to add dco sign-off checks if you prefer to have them

@aasseman aasseman merged commit 335896b into main Jul 21, 2023
3 checks passed
@aasseman aasseman deleted the aasseman/issue4 branch July 21, 2023 21:50
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@hopeyen hopeyen hopeyen approved these changes

Assignees

@aasseman aasseman

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

CI: tests and coverage
2 participants
@aasseman @hopeyen