Skip to content
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

ci: fix MSRV jobs with cargo update #159

Merged
merged 1 commit into from
Jul 28, 2023

Conversation

notmandatory
Copy link
Member

@notmandatory notmandatory commented Jul 26, 2023

Description

Use cargo update in the CI build test job to pin dependencies to work with MSRV 1.57.0. Also updated README with corresponding instructions.

Notes to the reviewers

This is how the rust-bitcoin and other bdk projects are going to be pinning dependencies from now on.

Changelog notice

None.

Checklists

All Submissions:

  • I've signed all my commits
  • I followed the contribution guidelines
  • I ran cargo fmt and cargo clippy before committing

@notmandatory notmandatory force-pushed the ci/fix_msrv branch 4 times, most recently from 5ff5859 to 646ab54 Compare July 26, 2023 01:40
@notmandatory notmandatory self-assigned this Jul 26, 2023
@notmandatory notmandatory added documentation Improvements or additions to documentation ci Github Actions CI Workflow Related labels Jul 26, 2023
@notmandatory
Copy link
Member Author

notmandatory commented Jul 26, 2023

@willowenss this PR fixes the bdk-cli MSRV dependencies issue. Please give it a review and an ACK if it looks OK to you and then I'll merge it.

After I merge it you'll need to rebase your branch for #156, let me know if you need help doing this.

@willowenss
Copy link

@willowenss this PR fixes the bdk-cli MSRV dependencies issue. Please give it a review and an ACK if it looks OK to you and then I'll merge it.

After I merge it you'll need to rebase your branch for #156, let me know if you need help doing this.

Sounds good

@notmandatory notmandatory merged commit fdb9d18 into bitcoindevkit:master Jul 28, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci Github Actions CI Workflow Related documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants