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

Move no-0-rent rent dist. behavior under feature#12804

Merged
mergify[bot] merged 1 commit intosolana-labs:masterfrom
ryoqun:no-0-rent-dist
Oct 12, 2020
Merged

Move no-0-rent rent dist. behavior under feature#12804
mergify[bot] merged 1 commit intosolana-labs:masterfrom
ryoqun:no-0-rent-dist

Conversation

@ryoqun
Copy link
Copy Markdown
Contributor

@ryoqun ryoqun commented Oct 12, 2020

Problem

#12791 introduces differing behavior.

Summary of Changes

Properly gate it. Fortunately, we have still not enabled runtime feature (no_overflow_rent_distribution); let's free-ride on it

Fixes #12791

@ryoqun ryoqun added automerge Merge this Pull Request automatically once CI passes v1.3 labels Oct 12, 2020
@ryoqun ryoqun requested a review from mvines October 12, 2020 07:38
@codecov
Copy link
Copy Markdown

codecov Bot commented Oct 12, 2020

Codecov Report

Merging #12804 into master will increase coverage by 0.0%.
The diff coverage is 100.0%.

@@           Coverage Diff           @@
##           master   #12804   +/-   ##
=======================================
  Coverage    81.9%    81.9%           
=======================================
  Files         360      360           
  Lines       84781    84781           
=======================================
+ Hits        69461    69474   +13     
+ Misses      15320    15307   -13     

@mergify mergify Bot merged commit 2f5bb7e into solana-labs:master Oct 12, 2020
mergify Bot pushed a commit that referenced this pull request Oct 12, 2020
mergify Bot pushed a commit that referenced this pull request Oct 12, 2020
mergify Bot added a commit that referenced this pull request Oct 12, 2020
(cherry picked from commit 2f5bb7e)

Co-authored-by: Ryo Onodera <ryoqun@gmail.com>
mergify Bot added a commit that referenced this pull request Oct 12, 2020
(cherry picked from commit 2f5bb7e)

Co-authored-by: Ryo Onodera <ryoqun@gmail.com>
@mvines
Copy link
Copy Markdown
Contributor

mvines commented Oct 12, 2020

Thank you @ryoqun, this was a close one!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

automerge Merge this Pull Request automatically once CI passes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants