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

Investigating bug in master #695

Draft
wants to merge 9 commits into
base: master
Choose a base branch
from

Conversation

cameronangliss
Copy link
Contributor

@cameronangliss cameronangliss commented Feb 7, 2025

I was able to recreate the bug I've been seeing in this log. I've only seen this bug pop up in concurrency tests, so that may be a clue. However, that also may just be a coincidence.

@cameronangliss cameronangliss changed the title expose bug Investigating concurrency bug Feb 7, 2025
Copy link

codecov bot commented Feb 7, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 85.16%. Comparing base (f458350) to head (0c7c49e).
Report is 124 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #695      +/-   ##
==========================================
+ Coverage   83.38%   85.16%   +1.78%     
==========================================
  Files          39       42       +3     
  Lines        3918     4409     +491     
==========================================
+ Hits         3267     3755     +488     
- Misses        651      654       +3     

@cameronangliss cameronangliss changed the title Investigating concurrency bug Investigating bug in master Feb 7, 2025
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