Skip to content

Add workflow for Minimum Rust Version badge #527

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 15 commits into from
Nov 9, 2021
Merged

Add workflow for Minimum Rust Version badge #527

merged 15 commits into from
Nov 9, 2021

Conversation

spenserblack
Copy link
Collaborator

@spenserblack spenserblack commented Nov 8, 2021

Resolves #525

This CI is very slow, taking about 15 minutes to complete. To help reduced conflicts, I added a git pull to the script that commits the badge.
However, this slowness might be something to consider for the workflow triggers. Perhaps the trigger should be on release instead?

@spenserblack spenserblack marked this pull request as ready for review November 8, 2021 14:55
@o2sh
Copy link
Owner

o2sh commented Nov 8, 2021

Perhaps the trigger should be on release instead?

Agreed 👍
I took care of it in e42dbb4 among other minor changes.

If the PR is good for you I'll let you trigger the workflow to generate a new badge before merging.

@spenserblack
Copy link
Collaborator Author

Changes look good! My only change after those was changing the literal + in the shields.io URL to be percent-encoded.
Badge generated.

@o2sh o2sh merged commit 04d0b01 into o2sh:main Nov 9, 2021
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.

Document minimum Rust version?
2 participants