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.

Sign up for GitHub

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

Jump to bottom

Fix tests coverage #1522

Merged
merged 10 commits into from
Jan 30, 2025
Merged

Fix tests coverage #1522

merged 10 commits into from
Jan 30, 2025

Conversation

peterMuriuki
Copy link
Collaborator

@peterMuriuki peterMuriuki commented Jan 20, 2025 •
edited
Loading

Changes included with this PR

  • Increase branch coverage to 80% to meet required threshold
  • update ci to pass even if windows side of matrix fails.

@peterMuriuki peterMuriuki marked this pull request as ready for review January 27, 2025 11:48
@peterMuriuki peterMuriuki merged commit 8ae64ef into main Jan 30, 2025
4 checks passed
@peterMuriuki peterMuriuki deleted the fix-tests-coverage branch January 30, 2025 07:20
@peterMuriuki peterMuriuki mentioned this pull request Feb 3, 2025
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@kahummer kahummer kahummer approved these changes

@machariamuguku machariamuguku Awaiting requested review from machariamuguku

@JohnMwashuma JohnMwashuma Awaiting requested review from JohnMwashuma

@ciremusyoka ciremusyoka Awaiting requested review from ciremusyoka

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@peterMuriuki @kahummer