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

Implement storage contract logic #2488

Merged
merged 1 commit into from
Jan 21, 2019

Conversation

sakridge
Copy link
Member

Problem

Validators need to be able to verify poreps and get paid.

Summary of Changes

Add transaction types that represent validators approving poreps and also advertising their storage last id.

Fixes #2203

@sakridge sakridge force-pushed the storage-contract-logic branch 2 times, most recently from 2fcbec9 to 4f727c3 Compare January 20, 2019 20:39
@codecov
Copy link

codecov bot commented Jan 20, 2019

Codecov Report

Merging #2488 into master will increase coverage by 0.3%.
The diff coverage is 87.2%.

@@           Coverage Diff            @@
##           master   #2488     +/-   ##
========================================
+ Coverage    77.4%   77.7%   +0.3%     
========================================
  Files         110     110             
  Lines       17716   18045    +329     
========================================
+ Hits        13725   14039    +314     
- Misses       3991    4006     +15

@sakridge sakridge force-pushed the storage-contract-logic branch 2 times, most recently from 4365fdd to 220c395 Compare January 20, 2019 22:28
Copy link
Contributor

@garious garious left a comment

Choose a reason for hiding this comment

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

Neato

@sakridge sakridge merged commit abbb037 into solana-labs:master Jan 21, 2019
@sakridge sakridge deleted the storage-contract-logic branch January 21, 2019 16:36
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.

3 participants