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 app auth #220

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

add app auth #220

wants to merge 1 commit into from

Conversation

kevinbuhmann
Copy link

@kevinbuhmann kevinbuhmann commented Apr 17, 2020

I'm not sure if this is something you all would want to add to the official checkout action or not. But I needed to use app auth in on one of my workflows so that I didn't need to create a personal access token with access to all of my personal private repos and add it to a work repo's secrets.

This PR is not complete. It's missing docs, tests, error handling, etc. But I can work on adding those things (with maybe a bit of guidance on the tests) if this is something that would get merged.

@kevinbuhmann kevinbuhmann changed the title feat: add app auth add app auth Apr 17, 2020
@kevinbuhmann
Copy link
Author

It looks the the CI is generating a different dist/index.js. Maybe that's b/c I'm using Windows?

@gle40059
Copy link

gle40059 commented Jun 7, 2020

(Hello-comment)

@kevinbuhmann kevinbuhmann changed the base branch from master to main September 15, 2020 13:48
@weiji14
Copy link

weiji14 commented Oct 28, 2020

Hi there 👋, just checking in to see if this still being actively worked on? This Github App authentication function would be very useful in an automated workflow I'm working on.

@loganvolkers
Copy link

🦗

@facundo-lancellotti-bm
Copy link

this will be very helpfull

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.

5 participants