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

Update to v0.12 #5614

Merged
merged 282 commits into from Jun 9, 2020
Merged

Update to v0.12 #5614

merged 282 commits into from Jun 9, 2020

Conversation

nisdas
Copy link
Member

@nisdas nisdas commented Apr 25, 2020

Do Not Merge:

@nisdas nisdas added the Blocked Blocked by research or external factors label Apr 25, 2020
@codecov
Copy link

codecov bot commented Apr 25, 2020

Codecov Report

Merging #5614 into master will decrease coverage by 0.00%.
The diff coverage is 56.91%.

@@            Coverage Diff             @@
##           master    #5614      +/-   ##
==========================================
- Coverage   59.87%   59.86%   -0.01%     
==========================================
  Files         323      323              
  Lines       27516    27550      +34     
==========================================
+ Hits        16474    16493      +19     
- Misses       8813     8815       +2     
- Partials     2229     2242      +13     

@nisdas nisdas marked this pull request as ready for review April 27, 2020 09:00
@nisdas nisdas requested a review from a team as a code owner April 27, 2020 09:00
@nisdas nisdas requested review from farazdagi, rauljordan and 0xKiwi and removed request for farazdagi, rauljordan and 0xKiwi April 27, 2020 09:00
@nisdas nisdas changed the title Update to v0.12 W.I.P: Update to v0.12 Apr 27, 2020
@nisdas nisdas changed the title W.I.P: Update to v0.12 WIP: Update to v0.12 Apr 27, 2020
@terencechain terencechain marked this pull request as draft April 27, 2020 13:51
@terencechain
Copy link
Member

Converted this to draft so even less likely someone review and merge it

@nisdas nisdas mentioned this pull request Jun 9, 2020
3 tasks
prylabs-bulldozer bot and others added 4 commits June 9, 2020 12:28
* rem outdated tests

* remove redundant

* benches passing

Co-authored-by: prylabs-bulldozer[bot] <58059840+prylabs-bulldozer[bot]@users.noreply.github.com>
beacon-chain/core/epoch/precompute/new.go Show resolved Hide resolved
beacon-chain/core/epoch/precompute/reward_penalty.go Outdated Show resolved Hide resolved
beacon-chain/sync/rpc_beacon_blocks_by_range.go Outdated Show resolved Hide resolved
shared/featureconfig/config.go Outdated Show resolved Hide resolved
rauljordan and others added 5 commits June 9, 2020 11:02
* Remove subscribeDynamic which was not dynamic anymore

* Add deployed Onyx contract

Co-authored-by: rauljordan <raul@prysmaticlabs.com>
* rem outdated tests

* remove redundant

* refactor and cleanup

* comment

* terence's review

* Update beacon-chain/rpc/validator/attester.go

* Update beacon-chain/rpc/validator/attester.go

* Update beacon-chain/rpc/validator/attester.go

* Update beacon-chain/rpc/validator/attester.go

* benches passing

* fix up attester test

Co-authored-by: rauljordan <raul@prysmaticlabs.com>
Co-authored-by: prylabs-bulldozer[bot] <58059840+prylabs-bulldozer[bot]@users.noreply.github.com>
Co-authored-by: terence tsao <terence@prysmaticlabs.com>
@prestonvanloon prestonvanloon changed the title WIP: Update to v0.12 Update to v0.12 Jun 9, 2020
@prestonvanloon prestonvanloon removed the Blocked Blocked by research or external factors label Jun 9, 2020
@rauljordan rauljordan mentioned this pull request Jun 9, 2020
rauljordan and others added 2 commits June 9, 2020 13:55
Copy link
Member

@terencechain terencechain left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Good work everyone!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Ready For Review A pull request ready for code review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants