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

Update workflows #338

Merged
merged 1 commit into from
Feb 5, 2024
Merged

Update workflows #338

merged 1 commit into from
Feb 5, 2024

Conversation

flodolo
Copy link
Contributor

@flodolo flodolo commented Feb 4, 2024

This PR includes changes from:

It also updates actions/cache to v4.

@flodolo
Copy link
Contributor Author

flodolo commented Feb 4, 2024

My goal here is to confirm that the workflows work, and have a log (the other two PRs are old enough that the logs have been deleted, and I couldn't figure out a way to re-run the actions without touching the forks).

Happy to merge those PRs to give credit where credit is due. The only addition here is updating actions/cache, because those deprecation warnings I could see in logs.

@flodolo flodolo removed the request for review from zbraniecki February 5, 2024 18:08
@flodolo
Copy link
Contributor Author

flodolo commented Feb 5, 2024

Thanks Greg, I'll merge the other 2 PRs and rebase this one.

This PR includes changes from:
- #332: Use -C rathen than -Z for instrument-coverage
- #320: Replace unmaintained actions-rs/toolchain with dtolnay/rust-toolchain@master

It also updates actions/cache to v4.
@flodolo flodolo merged commit 1d70202 into main Feb 5, 2024
4 checks passed
@flodolo flodolo deleted the update_workflows branch February 5, 2024 18:11
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