Fix the GitVersionPrerelease output#5
Conversation
|
Thanks, @g-galante It looks like a conflicting commit was added to main. Can you resolve the conflict? Then I think we can go ahead and merge this |
|
Heads up, OpenTAP gitversion behavior will change in the very near future due to this: opentap/opentap#1384 I think we should resolve build warnings, tag v1.1, update OpenTAP with the new behavior, and then tag v2.0 |
0c7aa05 to
f63daf1
Compare
|
Commit 42f0ef0, which you pushed in the morning, is conflicting with commit 0c7aa05, which I had pushed above, because they both modify actions' versions in The warnings in action Update setup-opentap (#7) #36 are coming from: andget-gitversion/.github/workflows/test.yml Line 25 in 42f0ef0 Do you mean tag The documentation has now been updated (commit 366e903) assuming you will tag it as |

Resolves #4
Commit 0c7aa05 takes care of the notice GitHub Actions: All Actions will run on Node16 instead of Node12 by default, upgrading all GitHub actions running on Node12 to a version running on Node16 to squelch the warnings.
The (last) remaining warning:

will disappear when opentap/setup-opentap#12 is merged and the reference in:
get-gitversion/action.yml
Line 19 in 1ecd47c
I volunteer to push one more commit to this PR to accomplish that when opentap/setup-opentap#12 is merged.