Skip to content

Conversation

@anorth
Copy link
Member

@anorth anorth commented Jun 10, 2024

I included the exported methods that are not part of an interface, since the sim really shouldn't be invoking these concurrently either.

Closes #304.

@anorth anorth requested a review from Stebalien June 10, 2024 21:41
@codecov
Copy link

codecov bot commented Jun 10, 2024

Codecov Report

Attention: Patch coverage is 58.62069% with 12 lines in your changes missing coverage. Please review.

Project coverage is 83.35%. Comparing base (8b8d26e) to head (09c4db2).

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #329      +/-   ##
==========================================
- Coverage   83.88%   83.35%   -0.53%     
==========================================
  Files          14       14              
  Lines        1613     1634      +21     
==========================================
+ Hits         1353     1362       +9     
- Misses        154      160       +6     
- Partials      106      112       +6     
Files Coverage Δ
gpbft/participant.go 83.14% <58.62%> (-5.39%) ⬇️

@anorth anorth added this pull request to the merge queue Jun 10, 2024
Merged via the queue into main with commit 462de82 Jun 10, 2024
@anorth anorth deleted the anorth/trylock branch June 10, 2024 23:41
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.

Consider protection against concurrent invocation of ReceiveMessage and ReceiveAlarm

3 participants