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

chore: Update geth 1.8.23 #4190

Merged
merged 1 commit into from
Mar 20, 2019
Merged

chore: Update geth 1.8.23 #4190

merged 1 commit into from
Mar 20, 2019

Conversation

evertonfraga
Copy link
Member

@evertonfraga evertonfraga changed the base branch from develop to master February 26, 2019 19:00
@evertonfraga evertonfraga changed the title Update geth 1.8.23 chore: Update geth 1.8.23 Feb 26, 2019
@nicksavers nicksavers mentioned this pull request Mar 1, 2019
@nicksavers
Copy link

@evertonfraga Why are you waiting to merge this?

@gnorbsl
Copy link

gnorbsl commented Mar 5, 2019

Can someone please approve and merge this. This shouldn't take that long after a fork that breaks chains.

@trashMeNow
Copy link

Hi devs.

I'm on windows 10 64 bit and Ethereum Wallet is my only wallet.
I don't want to use MEW or other third party wallets.
Plz, merge geth 1.8.23 to let Ethereum Wallet 0.11.1 usable with Costantinople/St. Petersburg main blockchain.

Thnak you all!

@rachhunter
Copy link

So users can keep up to date on the correct chain (now Constantinople/Petersburg), the update of Ethereum Wallet v0.11.1 to Geth v1.8.23 should have been available pre-fork.

Can this type of update please be a standard recurring workflow item, so that Mist/Ethereum Wallet are updated whenever Geth is updated and especially at times when a fork is imminent?

@gnorbsl
Copy link

gnorbsl commented Mar 19, 2019

That this hasn't been merged in 20 days says a lot about the project. Either add more people to the project who are able to review a PR or be more active. A update after a fork should not take three weeks if it only changes one file

@evertonfraga
@frozeman

@alexvandesande alexvandesande merged commit 0a0b7f8 into master Mar 20, 2019
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.

7 participants