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 update + update MSRV to 1.65.0 #79

Merged
merged 1 commit into from
May 10, 2024
Merged

CI update + update MSRV to 1.65.0 #79

merged 1 commit into from
May 10, 2024

Conversation

nastevens
Copy link
Member

This gets CI back into a working state. However, while making these updates I noticed that https://github.com/actions-rs is no longer maintained and it throws a ton of warnings. I'm opening this PR so we can unblock #77 and will open separate issues to migrate repos using actions-rs over to https://github.com/dtolnay/rust-toolchain.

@nastevens nastevens requested a review from a team May 7, 2024 02:26
@nastevens
Copy link
Member Author

Fixes #78

@nastevens
Copy link
Member Author

See rust-embedded/wg#754 tracking the actions-rs deprecation across the R-E org.

- armv7-unknown-linux-gnueabihf
- i686-unknown-linux-gnu
- i686-unknown-linux-musl
- mips-unknown-linux-gnu
Copy link
Contributor

@ryankurte ryankurte May 9, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks for the update! why is mips removed here?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

All mips targets are Tier 3 now, meaning they don't get official builds anymore.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Glad you had me look at this. I'd just copied this list from gpio-cdev without a critical eye. The loongarch64 is Tier 3 too, no reason to mention it in the list even commented out. And powerpc64 is tier 1 and would make sense to support. I'll fix this up.

@nastevens nastevens merged commit 6423dba into master May 10, 2024
93 checks passed
@nastevens nastevens deleted the ci-update branch May 10, 2024 20:06
@nastevens nastevens mentioned this pull request May 10, 2024
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