Skip to content

ci: Temporarily allow mismatched_lifetime_syntaxes lint in PACs #887

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

Merged
merged 1 commit into from
Jun 12, 2025

Conversation

jbeaurivage
Copy link
Contributor

Summary

mismatched_lifetime_syntaxes is a new lint introduced in rustc 1.84. PACs currently fail CI on nightly because of this.
Temporarily allow the lint. The #![allow] statement should be removed from update-pacs.sh when we start generating PACs with svd2rust >= 0.36.2: see rust-embedded/svd2rust#936 (currently unreleased).

@jbeaurivage jbeaurivage force-pushed the fix-pac-ci branch 2 times, most recently from be2675d to 1823422 Compare June 12, 2025 16:37
@ianrrees
Copy link
Contributor

@jbeaurivage I saw your messages in Rust Embedded about svd2rust - are you looking in to using a newer one?

@ianrrees ianrrees merged commit b6f1100 into atsamd-rs:master Jun 12, 2025
91 of 109 checks passed
@jbeaurivage jbeaurivage deleted the fix-pac-ci branch June 12, 2025 22:55
@jbeaurivage
Copy link
Contributor Author

@ianrrees ideally yes, I'm in no rush, but I felt like updating the pacs would be better than adding another lint exception. Not if it breaks the HAL though!

Looks like on nightly, we have some new dead_code warnings introduced - but those don't show up on stable. Weird, but we should probably figure out why to make ci green again

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