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 balance helpers v0.6 #2348

Merged

Conversation

shayzluf
Copy link
Contributor

[Part of] #2307


Add helper functions to balance change

includes:
get_balance
set_balance
increase_balance
decrease_balance

Write why you are making the changes in this pull request
to comply with 0.6 spec

Write a summary of the changes you are making
changes in:

@terencechain terencechain changed the base branch from master to spec-v0.6 April 23, 2019 03:53
@terencechain
Copy link
Member

updated base to spec-v0.6

@shayzluf shayzluf requested review from terencechain and rauljordan and removed request for terencechain April 23, 2019 04:52
@prestonvanloon
Copy link
Member

Please fix build issues

beacon-chain/core/helpers/rewards_penalties.go Outdated Show resolved Hide resolved
beacon-chain/core/helpers/rewards_penalties.go Outdated Show resolved Hide resolved
beacon-chain/core/helpers/rewards_penalties.go Outdated Show resolved Hide resolved
beacon-chain/core/helpers/rewards_penalties.go Outdated Show resolved Hide resolved
beacon-chain/core/helpers/rewards_penalties.go Outdated Show resolved Hide resolved
@shayzluf shayzluf force-pushed the update-balance-helpers-v0.6 branch from 748d4eb to 2e4a6a2 Compare April 24, 2019 05:34
@codecov
Copy link

codecov bot commented Apr 24, 2019

Codecov Report

Merging #2348 into spec-v0.6 will increase coverage by 0.13%.
The diff coverage is 100%.

@@              Coverage Diff              @@
##           spec-v0.6    #2348      +/-   ##
=============================================
+ Coverage      68.29%   68.42%   +0.13%     
=============================================
  Files            119      119              
  Lines           9260     9276      +16     
=============================================
+ Hits            6324     6347      +23     
+ Misses          2252     2247       -5     
+ Partials         684      682       -2

@shayzluf shayzluf requested a review from nisdas April 24, 2019 06:11
beacon-chain/core/helpers/rewards_penalties.go Outdated Show resolved Hide resolved
beacon-chain/core/helpers/rewards_penalties.go Outdated Show resolved Hide resolved
shared/params/config.go Outdated Show resolved Hide resolved
@shayzluf shayzluf force-pushed the update-balance-helpers-v0.6 branch from 8bafabf to 89de4ca Compare April 25, 2019 00:54
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.

👍 👍 👍

@terencechain terencechain merged commit 0035a5f into prysmaticlabs:spec-v0.6 Apr 25, 2019
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.

4 participants