-
-
Notifications
You must be signed in to change notification settings - Fork 62
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
upload assets on published release #117
Conversation
Not sure what you use the |
|
||
BINARY="$BASENAME.$GOOS.$GOARCH" | ||
BINARY_DIRECTORY="./bin/" | ||
BINARY_DIRECTORY="./bin" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The path had double slashes
see line 23 and 30.
We have access to all this event data: https://docs.github.com/en/free-pro-team@latest/developers/webhooks-and-events/webhook-events-and-payloads#release |
Co-authored-by: Joseph Petersen <josephp90@gmail.com>
- name: Build updateCli | ||
run: | | ||
touch updateCli.env | ||
make build.all |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Does the ubuntu-latest environment has golang installed?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Apparently yes - doc
Thanks, awesome, I wanted to work on this for a while
I use it source local environment variables, it's not needed anymore with your PR, I'll remove it |
Your very welcome! |
Noted the last two releases did not come with all the assets, so created this to upload them all on publish.