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

Commit

Permalink
Merge pull request #120 from ethereumjs/remove-old-fields
Browse files Browse the repository at this point in the history
remove old fields `consensus` and `finality`
  • Loading branch information
holgerd77 authored Jun 3, 2020
2 parents 3af85bc + 04be72a commit f15259a
Show file tree
Hide file tree
Showing 2 changed files with 9 additions and 27 deletions.
4 changes: 1 addition & 3 deletions lib/util/parse.js
Original file line number Diff line number Diff line change
Expand Up @@ -137,9 +137,7 @@ async function parseGethParams (json) {
}
params.hardforks = hardforks.map(name => ({
name: name,
block: name === 'chainstart' ? 0 : json.config[forkMap[name]] || null,
consensus: json.config.clique ? 'poa' : 'pow',
finality: name === 'hybridCasper' ? 'pos' : null
block: name === 'chainstart' ? 0 : json.config[forkMap[name]] || null
}))
return params
}
Expand Down
32 changes: 8 additions & 24 deletions test/util/params.json
Original file line number Diff line number Diff line change
Expand Up @@ -22,43 +22,27 @@
"bootstrapNodes": [],
"hardforks": [{
"name": "chainstart",
"block": 0,
"consensus": "poa",
"finality": null
"block": 0
}, {
"name": "homestead",
"block": 1,
"consensus": "poa",
"finality": null
"block": 1
}, {
"name": "dao",
"block": null,
"consensus": "poa",
"finality": null
"block": null
}, {
"name": "tangerineWhistle",
"block": 2,
"consensus": "poa",
"finality": null
"block": 2
}, {
"name": "spuriousDragon",
"block": 3,
"consensus": "poa",
"finality": null
"block": 3
}, {
"name": "byzantium",
"block": 1035301,
"consensus": "poa",
"finality": null
"block": 1035301
}, {
"name": "constantinople",
"block": null,
"consensus": "poa",
"finality": null
"block": null
}, {
"name": "hybridCasper",
"block": null,
"consensus": "poa",
"finality": "pos"
"block": null
}]
}

0 comments on commit f15259a

Please sign in to comment.