Skip to content
This repository was archived by the owner on Jan 22, 2025. It is now read-only.

Make sure banking stage is recording with the same bank that it read #3447

Merged
merged 3 commits into from
Mar 22, 2019

Conversation

aeyakovenko
Copy link
Member

Problem

Poh recorder could have a new working bank.

Summary of Changes

Make sure banking stage is recording with the same bank that it read with.

Fixes #

@aeyakovenko aeyakovenko changed the title make sure banking stage is recording with the same bank that it read … Make sure banking stage is recording with the same bank that it read Mar 22, 2019
@aeyakovenko aeyakovenko requested review from sakridge and pgarg66 March 22, 2019 20:31
@aeyakovenko aeyakovenko requested a review from rob-solana March 22, 2019 20:38
@codecov
Copy link

codecov bot commented Mar 22, 2019

Codecov Report

Merging #3447 into master will increase coverage by <.1%.
The diff coverage is 100%.

@@           Coverage Diff            @@
##           master   #3447     +/-   ##
========================================
+ Coverage    79.9%   79.9%   +<.1%     
========================================
  Files         148     148             
  Lines       23291   23320     +29     
========================================
+ Hits        18617   18646     +29     
  Misses       4674    4674

1 similar comment
@codecov
Copy link

codecov bot commented Mar 22, 2019

Codecov Report

Merging #3447 into master will increase coverage by <.1%.
The diff coverage is 100%.

@@           Coverage Diff            @@
##           master   #3447     +/-   ##
========================================
+ Coverage    79.9%   79.9%   +<.1%     
========================================
  Files         148     148             
  Lines       23291   23320     +29     
========================================
+ Hits        18617   18646     +29     
  Misses       4674    4674

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants