getporter / porter

Porter enables you to package your application artifact, client tools, configuration and deployment logic together as an installer that you can distribute, and install with a single command.

https://porter.sh

Apache License 2.0

1.2k stars 199 forks source link

Checkout history when cross compiling in the canary workflow #3131

Closed kichristensen closed 2 months ago

kichristensen commented 2 months ago

What does this change

Otherwise the Git version calculated will not match the one used later when publishing the binaries.

What issue does it fix

Hopefully handles #3073

Notes for the reviewer

It is really hard to test without merging, because it is using different code flows depending on the branch name

Checklist

[ ] Did you write tests?
[ ] Did you write documentation?
[ ] Did you change porter.yaml or a storage document record? Update the corresponding schema file.
[ ] If this is your first pull request, please add your name to the bottom of our Contributors list. Thank you for making Porter better! 🙇‍♀️