Skip to content

Make option_channel_type mandatory#3046

Merged
remyers merged 1 commit into
ACINQ:masterfrom
remyers:mandatory_channel_type
Mar 26, 2025
Merged

Make option_channel_type mandatory#3046
remyers merged 1 commit into
ACINQ:masterfrom
remyers:mandatory_channel_type

Conversation

@remyers
Copy link
Copy Markdown
Contributor

@remyers remyers commented Mar 26, 2025

This is the first step before we make the change to always assume option_channel_type.

See lightning/bolts#1232

Removed tests that depend on a node without the option_channel_type feature set.

Updated some tests that must explicitly set the init features to include option_channel_type.

@remyers remyers requested a review from t-bast March 26, 2025 10:34
This is the first step before we make the change to always assume option_channel_type.

see lightning/bolts#1232

Removed tests that depend on a node without the option_channel_type feature set.

Updated some tests that must explicitly set the init features to include option_channel_type.
@remyers remyers force-pushed the mandatory_channel_type branch from 5563613 to 2b77e7e Compare March 26, 2025 10:40
Copy link
Copy Markdown
Member

@t-bast t-bast left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@remyers remyers merged commit 2dfdc26 into ACINQ:master Mar 26, 2025
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.

2 participants