Skip to content

fix: cleanup CI workflow#108

Merged
nyonson merged 4 commits intorust-bitcoin:mainfrom
nyonson:fix/loosen-bitcoin-dep
May 14, 2025
Merged

fix: cleanup CI workflow#108
nyonson merged 4 commits intorust-bitcoin:mainfrom
nyonson:fix/loosen-bitcoin-dep

Conversation

@nyonson
Copy link
Collaborator

@nyonson nyonson commented May 14, 2025

  • First patch has lint fixes. There isn't an easy way to fix/ignore the ones in the generated code of configure_me, but the blast radius is minimized to just the proxy bin which I don't care much about.
  • Second patch folds in some of the CI lessons from kyoto, is mostly a shifting around of things. I did have to bump corepc-node and now explicitly set the download flag.

@nyonson nyonson force-pushed the fix/loosen-bitcoin-dep branch 3 times, most recently from 65aad4a to 90a8411 Compare May 14, 2025 18:31
@nyonson nyonson force-pushed the fix/loosen-bitcoin-dep branch 6 times, most recently from ec7baea to 2017810 Compare May 14, 2025 20:19
@nyonson nyonson marked this pull request as ready for review May 14, 2025 20:25
@nyonson nyonson requested a review from rustaceanrob May 14, 2025 20:25
@nyonson nyonson force-pushed the fix/loosen-bitcoin-dep branch 2 times, most recently from fb36e28 to db83049 Compare May 14, 2025 22:20
nyonson added 3 commits May 14, 2025 15:22
This library will probably go "cold" soon since it is feature complete,
so the CI pipe won't be triggered often. But would like to be alerted
to breaks.
@nyonson nyonson force-pushed the fix/loosen-bitcoin-dep branch from db83049 to 13dede9 Compare May 14, 2025 22:22
@nyonson nyonson merged commit e104647 into rust-bitcoin:main May 14, 2025
10 checks passed
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.

1 participant