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

bump proc-macro2 dep to pass nightly tests and add fail-fast-false #169

Merged
merged 2 commits into from
Feb 10, 2025

Conversation

miguelraz
Copy link
Contributor

@miguelraz miguelraz commented Feb 10, 2025

This PR involves 2 small changes:

  1. It bumps the proc-macro2 dependency to one that doesn't break on nightly (This was a well documented break then.)
  2. It adds a fail-fast: false argument to the weekly canary to make it so if one stable/beta/nightly fails the other jobs still run.

Should fix #168

Copy link

cloudflare-workers-and-pages bot commented Feb 10, 2025

Deploying ferrous-systems-rust-exercises with  Cloudflare Pages  Cloudflare Pages

Latest commit: ae22e68
Status: ✅  Deploy successful!
Preview URL: https://94af105d.ferrous-systems-rust-exercises.pages.dev
Branch Preview URL: https://canary-fail-fast-false.ferrous-systems-rust-exercises.pages.dev

View logs

@jonathanpallant
Copy link
Member

Did you run cargo update -p proc-macro2 or did you manually edit Cargo.lock?

@miguelraz miguelraz merged commit 923d14e into main Feb 10, 2025
3 checks passed
@miguelraz
Copy link
Contributor Author

Manually, I didn't want to pull in the most recent version because that may have introduced more breaking changes, so I wanted a lower version if possible.

@miguelraz miguelraz deleted the canary-fail-fast-false branch February 10, 2025 16:49
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.

Nightly failure on proc-macros2?
2 participants