Skip to content

Conversation

@TheBlueMatt
Copy link
Collaborator

Fixes the docs.rs build but also fixes _test_utils channel opens which I need for a splicing upgrade test I want to land for 0.2.

These have been merged, causing our docs.rs builds to fail. Sadly,
we saw our docs.rs build fail for the 0.1.6 upload because of this.

Backport of 6aea586

Resolved conflicts in:
 * lightning-background-processor/src/lib.rs
 * lightning-liquidity/src/lib.rs
 * lightning/src/lib.rs
Sadly, our docs.rs build failed for 0.1.6 due to the unstable
`doc_auto_cfg` feature being removed. Here, we try to emulate
docs.rs builds as best we can in CI to head that off in the future.

Backport of d3faa03
In our upgrade/downgrade tests we want to be able to use the
functional test utils as-is externally as a part of CI in later
LDK versions. This implies that code that is `cfg(test)` really
needs to always be `cfg(any(test, feature = "_test_utils"))`.

Sadly, trying to open multiple channels back-to-back on a node
with `_test_utils` currently fails as we re-broadcast channel
announcement messages, something the functional test utils does not
expect.

Here we update the `cfg`-flagging of that rebroadcast logic to fix
back-to-back channel opens in `_test_utils`.

This narrowly backports one hunk of the much larger upstream
commit e41e756.
@ldk-reviews-bot
Copy link

ldk-reviews-bot commented Oct 21, 2025

👋 Thanks for assigning @valentinewallace as a reviewer!
I'll wait for their review and will help manage the review process.
Once they submit their review, I'll check if a second reviewer would be helpful.

Copy link
Contributor

@tankyleo tankyleo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

./ci/ci-tests.sh passed on my machine

@ldk-reviews-bot
Copy link

👋 The first review has been submitted!

Do you think this PR is ready for a second reviewer? If so, click here to assign a second reviewer.

@TheBlueMatt TheBlueMatt merged commit 4c1aa13 into lightningdevkit:0.1 Oct 21, 2025
20 of 26 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.

5 participants