Skip to content

CI: Use correct MSRV toolchain#5351

Merged
apoelstra merged 1 commit intorust-bitcoin:0.32.xfrom
tcharding:push-yvmzzsmzvsmu
Dec 1, 2025
Merged

CI: Use correct MSRV toolchain#5351
apoelstra merged 1 commit intorust-bitcoin:0.32.xfrom
tcharding:push-yvmzzsmzvsmu

Conversation

@tcharding
Copy link
Copy Markdown
Member

@tcharding tcharding commented Nov 30, 2025

In commit 2a40a35 (PR #3418) I copied the rust.yml file from master and in doing so used the wrong MSRV toolchain.

Use the correct MSRV toolchain for this branch. This means that since October 2024 we have not been testing in CI with the advertised MSRV toolchain.

Fix: #5350

In commit 2a40a35 (PR rust-bitcoin#3418) I copied the `rust.yml` file from
`master` and in doing so used the wrong MSRV toolchain.

Use the correct MSRV toolchain for this branch.
@github-actions github-actions bot added the ci label Nov 30, 2025
@tcharding
Copy link
Copy Markdown
Member Author

@apoelstra has your CI setup been using the correct one? I.e are your scripts totally custom or do you somehow base them off of rust.yml?

@tcharding
Copy link
Copy Markdown
Member Author

tcharding commented Nov 30, 2025

GitHub is not picking up that this PR closes an issue, second time I've seen that bug recently.

EDIT: I raised a ticket: https://support.github.com/ticket/personal/0/3932091

@apoelstra
Copy link
Copy Markdown
Member

I.e are your scripts totally custom or do you somehow base them off of rust.yml?

My stuff reads clippy.toml.

@tcharding
Copy link
Copy Markdown
Member Author

Phew so we were covered all this time. Great. Thanks man.

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 3e4c121; successfully ran local tests; since MSRV for this branch won't ever change, but really we ought to somehow get a SSOT here

@apoelstra apoelstra merged commit 86c6960 into rust-bitcoin:0.32.x Dec 1, 2025
19 of 20 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants