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

Add github builtin authorization to support private repos #7

Merged
merged 1 commit into from
Mar 21, 2022
Merged

Conversation

fiskhest
Copy link
Contributor

@fiskhest fiskhest commented Mar 21, 2022

Per my own testing and #5 it seems like this action currently only works for public repositories. I needed it in a private repository, and the fix is real simple.

Whenever Github Actions is enabled for a repository, a Github Apps is installed and the ${{ github.token }} variable is exposed.

The proposed changes piggybacks on the fact that variable will always be populated and thus we can lookup the workflow status for both private and public repositories.

@martialonline martialonline self-requested a review March 21, 2022 12:19
@martialonline
Copy link
Owner

Thanks for the PR @fiskhest! The Action was originally developed for public repositories but the token addition makes it compatible with private ones too. 👍🏻

@martialonline martialonline merged commit 616a964 into martialonline:master Mar 21, 2022
@fiskhest
Copy link
Contributor Author

🙏👍 Cheers for the quick response! 🎉

@fiskhest
Copy link
Contributor Author

fiskhest commented Mar 22, 2022

@martialonline I might've messed something up with the PR, but a new tag was never built so there's no v3 tag pointing to the latest commit (I'm currently bypassing this by checking out master instead but would prefer a tag). Could you possibly have a look?

@martialonline
Copy link
Owner

@fiskhest all done - published the v3 release. Sorry it took a while!

@fiskhest
Copy link
Contributor Author

fiskhest commented Apr 5, 2022

No worries, thank you!

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.

2 participants