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

1 preview configs #887

Merged
merged 3 commits into from
Jun 19, 2019
Merged

1 preview configs #887

merged 3 commits into from
Jun 19, 2019

Conversation

josedahlquist
Copy link
Collaborator

No description provided.

@josedahlquist josedahlquist marked this pull request as ready for review June 18, 2019 18:52
@colltoaction
Copy link
Contributor

I left a TODO here: https://github.com/rsksmart/rskj/blob/master/rskj-core/src/main/java/org/ethereum/config/blockchain/upgrades/ConsensusRule.java#L23

I was hoping we would tackle it as part of the PR that created a new testnet.

Also, shouldn't we change the Genesis, the network id or anything?

@josedahlquist josedahlquist force-pushed the 1-preview-configs branch 4 times, most recently from 5b6b4ac to 07f76f4 Compare June 18, 2019 19:55
lsebrie
lsebrie previously approved these changes Jun 18, 2019
colltoaction
colltoaction previously approved these changes Jun 18, 2019
pmprete
pmprete previously approved these changes Jun 19, 2019
Copy link

@pmprete pmprete left a comment

Choose a reason for hiding this comment

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

LGTM 👍

diega
diega previously approved these changes Jun 19, 2019
pmprete
pmprete previously approved these changes Jun 19, 2019
Copy link

@pmprete pmprete left a comment

Choose a reason for hiding this comment

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

👍

@diega diega dismissed stale reviews from pmprete and themself via ff5721a June 19, 2019 14:38
Copy link
Contributor

@diega diega left a comment

Choose a reason for hiding this comment

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

LGTA

@diega diega merged commit f1f4554 into master Jun 19, 2019
@diega diega deleted the 1-preview-configs branch June 19, 2019 14:46
@aeidelman aeidelman added this to the Wasabi v1.0.0 milestone Jul 5, 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.

7 participants