-
-
Notifications
You must be signed in to change notification settings - Fork 0
Add GitHub release automation from tags #3
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
Conversation
e239952 to
6b67bd5
Compare
Maybe it's worth to switch to the v0.9.2 tagging scheme? I think it's much more popular across Rust crates. |
|
I only really see disadvantages of the |
|
The job isn't running on newly pushed tags which should match the given pattern. Do you know how to debug this? |
|
It looks like you've missed one indentation level after line 25, which causes the following error:
|
|
Hm, no. I was wrong. The config in master looks correct. |
|
The regexp may be incorrect. I get the following error for |
|
Also note that your forgot the |
|
This is some form of glob pattern not a regexp. That's part of the problem. No, I didn't publish; I just pushed a couple of old tags. |
|
Try to remove |
|
No dice. I'll try with a 'v' prefix. |
|
I pushed v0.0.0 tag to the latest master commit and it triggered the workflow. So it seems it may not work on older commits for some reason. |
|
Oh... I wondered where that came from. |
|
I think I understand why the workflow does not get triggered. GH looks for configs in the commit itself, so if we tag an older commit, GH thinks that there is no such workflow. |
|
Confirmed: tagging |
|
BTW release names (i.e "release_0.10.0-rc-1") look a bit ugly in my opinion. |
@newpavlov @josephlr this is a cut-down version of rust-random/getrandom#740 with only the GitHub release automation.
This assumes tags of the form "0.9.2" (no leading 'v'), which is what we've been using in rand. I have such tags from past releases ready to push after this is merged.