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

Fixed kovan chain validation #6758

Merged
merged 3 commits into from
Oct 14, 2017
Merged

Fixed kovan chain validation #6758

merged 3 commits into from
Oct 14, 2017

Conversation

arkpar
Copy link
Collaborator

@arkpar arkpar commented Oct 14, 2017

No description provided.

@arkpar arkpar added A0-pleasereview 🤓 Pull request needs code review. M4-core ⛓ Core client code / Rust. labels Oct 14, 2017
@arkpar arkpar requested a review from rphmeier October 14, 2017 17:51
@5chdn 5chdn added this to the 1.8 milestone Oct 14, 2017
@5chdn 5chdn added the B0-patch label Oct 14, 2017
@@ -85,6 +85,8 @@ pub struct CommonParams {
pub eip155_transition: BlockNumber,
/// Validate block receipts root.
pub validate_receipts_transition: BlockNumber,
/// Validate trsanasction chain id.
Copy link
Contributor

Choose a reason for hiding this comment

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

typo: trsanasc

@rphmeier rphmeier added A6-mustntgrumble 💦 Pull request has areas for improvement. The author need not address them before merging. and removed A0-pleasereview 🤓 Pull request needs code review. labels Oct 14, 2017
@rphmeier
Copy link
Contributor

other than typo LGTM, assuming it syncs

@arkpar arkpar added A0-pleasereview 🤓 Pull request needs code review. and removed A6-mustntgrumble 💦 Pull request has areas for improvement. The author need not address them before merging. labels Oct 14, 2017
@keorn keorn added A8-looksgood 🦄 Pull request is reviewed well. and removed A0-pleasereview 🤓 Pull request needs code review. labels Oct 14, 2017
@arkpar arkpar merged commit b0c1549 into master Oct 14, 2017
@arkpar arkpar deleted the kovan-sync-fix branch October 14, 2017 19:59
arkpar added a commit that referenced this pull request Oct 14, 2017
* Fixed kovan chain validation

* Fork detection

* Fixed typo
arkpar added a commit that referenced this pull request Oct 14, 2017
* Fixed kovan chain validation

* Fork detection

* Fixed typo
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A8-looksgood 🦄 Pull request is reviewed well. M4-core ⛓ Core client code / Rust.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants