Repository navigation
Release 4.1.7 - #34
Merged
Merged
Conversation
Everything the field test turned up: fee slider steps the miner will accept (#23), the fiat display no longer raising a TypeError per rendered amount and no longer showing dollars under a euro label (#16, #17, #18), only sources that quote this chain (#24), a description column that can be dragged (#20), four bundled servers in four buckets rather than three (#22), and Windows resolving its own names again (#19). First release that went through CI -- there has been none here since 2021. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Version bump for the 4.1.7 release.
electrum/version.pyonly — nothing else in the tree hardcodes the version.What 4.1.7 carries
Everything the Windows field test turned up, plus what came out of checking it:
fee_per_kb()started clamping itTypeErrorper rendered amountFirst release that went through CI
There had been none since July 2021 (#33).
mainnow runs flake8 and 183 unit tests on every push.Two latent breakages surfaced while setting that up and are fixed in this release:
protobufhad no upper bound, so a freshpip install -r contrib/requirements/requirements.txtproduced a package that could not parse a payment request; and the fiat defect had madetest_wallet.pyhang rather than fail, which is why nobody had seen it.Still open, deliberately
server.peers.subscribereturns empty, so nothing is discovered. Which fleet hosts become public defaults is an operational decision0, not 7070🤖 Generated with Claude Code