Skip to content

Add new api text file changes#3721

Merged
apoelstra merged 1 commit intorust-bitcoin:masterfrom
tcharding:12-11-api-text-files
Dec 12, 2024
Merged

Add new api text file changes#3721
apoelstra merged 1 commit intorust-bitcoin:masterfrom
tcharding:12-11-api-text-files

Conversation

@tcharding
Copy link
Copy Markdown
Member

@tcharding tcharding commented Dec 10, 2024

In a recent PR (#3682) we introduced api text files. Then in another PR (#3711) we removed alloc feature gating. Possibly due to the timing of running through CI and merging these two PRs managed to get merged without an update to the API text files.

As would be expected; removing the alloc feature gate adds a bunch of new lines to the no-features api text file.

In a recent PR (rust-bitcoin#3682) we introduced api text files. Then in another
PR (rust-bitcoin#3711) we removed `alloc` feature gating. Possibly due to the
timing of running through CI and merging these two PRs managed to get
merged without an update to the API text files.

As would be expected; removing the `alloc` feature gate adds a bunch of
new lines to the `no-features` api text file.
Copy link
Copy Markdown
Contributor

@jamillambert jamillambert left a comment

Choose a reason for hiding this comment

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

ACK 601a47f

@tcharding
Copy link
Copy Markdown
Member Author

bump @apoelstra can you prioritise this one please, its blocking all other open PRs.

Copy link
Copy Markdown
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 601a47f; successfully ran local tests

@apoelstra apoelstra merged commit aeca93b into rust-bitcoin:master Dec 12, 2024
@tcharding tcharding deleted the 12-11-api-text-files branch December 12, 2024 23:46
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.

3 participants