Skip to content

CHANGELOG.md: put missing CHANGELOG entries in for PR 9136.#9160

Merged
rustyrussell merged 1 commit into
ElementsProject:masterfrom
rustyrussell:late-deprecated-removals
May 21, 2026
Merged

CHANGELOG.md: put missing CHANGELOG entries in for PR 9136.#9160
rustyrussell merged 1 commit into
ElementsProject:masterfrom
rustyrussell:late-deprecated-removals

Conversation

@rustyrussell
Copy link
Copy Markdown
Contributor

Removals should be documented with Changelog-Removed lines, mirroring the old Deprecated line.

In this case, the max_total_htlc_in_msat removal was premature: we try to give one version AFTER the deprecation period.

That way users can re-enable the feature if they need to, using --i-promise-to-fix-broken-api-user=listpeerchannels.max_total_htlc_in_msat and we can decide whether to defer removal, re-enable, even un-deprecate (or get whatever tool they are using fixed).

It's happened a few times in the past that nobody tested with deprecations disabled, and users got stuck when they upgraded :(
Changelog-None

Removals should be documented with Changelog-Removed lines, mirroring
the old Deprecated line.

In this case, the `max_total_htlc_in_msat` removal was premature: we try
to give one version AFTER the deprecation period.

That way users can re-enable the feature if they need to, using
`--i-promise-to-fix-broken-api-user=listpeerchannels.max_total_htlc_in_msat`
and we can decide whether to defer removal, re-enable, even un-deprecate
(or get whatever tool they are using fixed).

It's happened a few times in the past that nobody tested with deprecations
disabled, and users got stuck when they upgraded :(

Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
@rustyrussell rustyrussell requested a review from daywalker90 May 21, 2026 01:51
@rustyrussell rustyrussell merged commit 2b8684f into ElementsProject:master May 21, 2026
39 of 43 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant