Skip to content

Conversation

@RCasatta
Copy link
Collaborator

The elip rename is a breaking change, but it's highly likely we are the only one using it and a patch release would allow us to avoid the deps release dance

@LeoComandini
Copy link
Contributor

utACK f2ffa00, code review

I think no one was using the constant in the elip101/2 module, otherwise they would have notice the bug fixed in #224... so agree I think a patch release is fine

@apoelstra
Copy link
Member

Yeah, agreed. I'll go ahead and release this.

Copy link
Member

@apoelstra apoelstra left a comment

Choose a reason for hiding this comment

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

ACK f2ffa00; successfully ran local tests

@apoelstra apoelstra merged commit 378e753 into ElementsProject:master Apr 18, 2025
5 of 6 checks passed
@apoelstra
Copy link
Member

Tagged and published.

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.

4 participants