Skip to content
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

Fix GitHub token in android upload release action #76

Conversation

francisduvivier
Copy link
Contributor

@francisduvivier francisduvivier commented Oct 5, 2020

Hi, very small PR here because I realized a mistake in my previous PR(#74 ).
The android apk upload would not work if the GITHUB_TOKEN as well as the GH_TOKEN exist, so I removed the GITHUB_TOKEN usage.
This is also more in line with the the other yml files in the repo.

I added GH_TOKEN in my fork and created a release to test the action run.

Also, I saw I used CRLF line endings, so I changed that to LF.

This GITHUB_TOKEN was previously included with the assumption that a repo either has GH_TOKEN or GITHUB_TOKEN, but it is actually possibly that both will be available, which will would make the upload fail.
Also, the other actions in the repo just use GH_TOKEN.
@thias15 thias15 merged commit 7f168f0 into isl-org:master Oct 5, 2020
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.

None yet

2 participants