Skip to content
This repository has been archived by the owner on Apr 6, 2020. It is now read-only.

[MASTER-RELEASE] Byzantium Release (needs dependency update) #34

Merged
merged 2 commits into from
Sep 25, 2017

Conversation

holgerd77
Copy link
Member

No description provided.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 64.078% when pulling dc7e742 on byzantium-release into c577253 on master.

@jwasinger jwasinger self-requested a review September 21, 2017 23:34
package.json Outdated
@@ -26,7 +26,7 @@
"dependencies": {
"async": "^2.1.2",
"ethashjs": "~0.0.7",
"ethereumjs-block": "~1.6.0",
"ethereumjs-block": "git+https://github.com/ethereumjs/ethereumjs-block.git#byzantium-changes",
Copy link
Contributor

Choose a reason for hiding this comment

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

Once we release ethereumjs-block Byzantium changes, we will change this line to point to the master branch of ethereumjs-block. Later, after we successfully release, then we can release a new version of ethereumjs-block and change this line again to restrict the version.

@holgerd77 holgerd77 changed the title [WIP] Byzantium Release, don't merge yet [MASTER-RELEASE] Byzantium Release (needs dependency update9 Sep 25, 2017
@holgerd77 holgerd77 changed the title [MASTER-RELEASE] Byzantium Release (needs dependency update9 [MASTER-RELEASE] Byzantium Release (needs dependency update) Sep 25, 2017
Copy link
Member Author

@holgerd77 holgerd77 left a comment

Choose a reason for hiding this comment

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

Needs to be pointed to master after having merged ethereumjs-block, otherwise looks good.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 64.078% when pulling 3eacb63 on byzantium-release into c577253 on master.

@cdetrio cdetrio merged commit ff2594e into master Sep 25, 2017
@jwasinger jwasinger deleted the byzantium-release branch September 25, 2017 18:55
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants