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

Add tests that catch a few more edge cases #210

Merged
merged 1 commit into from
Aug 26, 2023
Merged

Add tests that catch a few more edge cases #210

merged 1 commit into from
Aug 26, 2023

Conversation

charleskawczynski
Copy link
Owner

These added tests catch a few more edge cases, related to #200, but they don't require running an entire tournament.

@charleskawczynski
Copy link
Owner Author

bors r+

@codecov
Copy link

codecov bot commented Aug 26, 2023

Codecov Report

Merging #210 (6d2da6f) into main (e25418d) will decrease coverage by 0.16%.
Report is 2 commits behind head on main.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main     #210      +/-   ##
==========================================
- Coverage   82.33%   82.18%   -0.16%     
==========================================
  Files          13       13              
  Lines        1319     1319              
==========================================
- Hits         1086     1084       -2     
- Misses        233      235       +2     

see 2 files with indirect coverage changes

@bors
Copy link
Contributor

bors bot commented Aug 26, 2023

Build succeeded!

The publicly hosted instance of bors-ng is deprecated and will go away soon.

If you want to self-host your own instance, instructions are here.
For more help, visit the forum.

If you want to switch to GitHub's built-in merge queue, visit their help page.

@bors bors bot merged commit b01e336 into main Aug 26, 2023
20 of 21 checks passed
@bors bors bot deleted the ck/more_tests branch August 26, 2023 21:55
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.

1 participant