Skip to content

Fix recent "alloc" feature fail #87

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 2 commits into from
Mar 1, 2023

Conversation

tcharding
Copy link
Member

@tcharding tcharding commented Mar 1, 2023

Recently in #71 I introduced a bucket load of bugs, in CI and in the with-allocator crate (which was not being run correctly in CI).

  • Patch 1 enables the "alloc" feature in the with-allocator crate.
  • Patch 2 totally overhauls the github actions

@tcharding tcharding mentioned this pull request Mar 1, 2023
As the crate name suggests `with_allocator` requires the "alloc" feature
to be enabled.
@tcharding tcharding force-pushed the 03-01-fix-alloc branch 6 times, most recently from 764ffba to d8c3bc6 Compare March 1, 2023 03:26
Rework the github actions to fix the many bugs Tobin recently
introduced.

- Use dtolnay instead of actions-rs (non-embedded only)
- Put fmt and clippy control into the CI script
@tcharding
Copy link
Member Author

tcharding commented Mar 1, 2023

Phew, that was harder than it should have been.

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 7873c85

@apoelstra apoelstra merged commit f33d2b0 into rust-bitcoin:master Mar 1, 2023
@tcharding tcharding mentioned this pull request Mar 1, 2023
@tcharding tcharding deleted the 03-01-fix-alloc branch March 1, 2023 23:50
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