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 workflow actions to latest #121

Merged
merged 2 commits into from
Aug 11, 2023

Conversation

bgilbert
Copy link
Contributor

@bgilbert bgilbert commented Aug 2, 2023

Fix various deprecation warnings. The actions-rs and actions/*release* workflows are unmaintained; replace the former with manual Rust commands and the latter with a substitute.

Use generic content type for uploaded artifacts. This avoids the need to run the release step once per artifact, and the content types don't seem to be attached to the final download anyway.

Also drop monthly run of Rust workflow. We already have the cargo update PR to ensure there's a monthly CI run. Removing the timer from the Rust job allows it to be enabled in forks for pre-PR testing without also running it periodically there.

Copy link
Member

@cecton cecton left a comment

Choose a reason for hiding this comment

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

All good but same comment than mbrman: rust-disk-partition-management/mbrman#28 (comment) I'm not too fan of actions for things that look so simple

We already have the `cargo update` PR to ensure there's a monthly CI run.
Removing the timer from the Rust job allows it to be enabled in forks for
pre-PR testing without also running it periodically there.
Fix various deprecation warnings.  The actions-rs and actions/*release*
workflows are unmaintained; replace the former with manual Rust commands
and the latter with a substitute.

Use generic content type for uploaded artifacts.  This avoids the need to
run the release step once per artifact, and the content types don't seem
to be attached to the final download anyway.
@bgilbert
Copy link
Contributor Author

bgilbert commented Aug 4, 2023

Okay, updated!

@bgilbert bgilbert marked this pull request as ready for review August 4, 2023 02:30
@bgilbert bgilbert merged commit 671b323 into rust-disk-partition-management:main Aug 11, 2023
9 checks passed
@bgilbert bgilbert deleted the workflows branch August 11, 2023 16:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants