Skip to content

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