pull down to refresh

This Week in Bitcoin Core there were a few changes merged in addition to a new maintainer TheCharlatan aka sedited.

Merged ChangesMerged Changes

  • wallet: fix unnamed legacy wallet migration failure by furszy
  • wallet: warn against accidental unsafe older() import by Sjors
  • refactor: Let CCoinsViewCache::BatchWrite return void by sedited

Change needing reviewChange needing review

  • wallet: make migration more robust against failures

Be sure to read the article for the full scoopBe sure to read the article for the full scoop

this is misleading. a new maintainer was nominated. see #1407587

reply

The post you're linking to is a post written by me.

TheCharlatan aka sedited was nominated and selected yesterday via the bitcoin dev weekly IRC meeting.

reply

my bad. the "A bunch of Acks" means he was not only nominated, but also selected?

reply

No problem.

Yea it was general consensus that he will be the next maintainer, to make it official he needs to add his PGP keys to the maintainer list on GitHub.

From what I understand it is up to the current maintainers to add a new maintainer and they were on the IRC meeting or were consulted prior.

But it is technically not official till his keys are added. Similar to an elected official taking the oath.

reply
21 sats \ 1 reply \ @nathanael 13h

thanks for the clarification. sorry that i nitpicked... i am working on that

reply

Getting the details right is a core quality of Bitcoiners!

Don't be sorry about getting the details right! I'd rather know the truth than assume something incorrectly as the truth.

reply