Skip to content
This repository was archived by the owner on Dec 5, 2022. It is now read-only.

Add note about required permissions on PAT#17

Merged
maxheld83 merged 1 commit intomaxheld83:masterfrom
alexlafroscia:alexlafroscia-patch-1
Sep 25, 2019
Merged

Add note about required permissions on PAT#17
maxheld83 merged 1 commit intomaxheld83:masterfrom
alexlafroscia:alexlafroscia-patch-1

Conversation

@alexlafroscia
Copy link
Copy Markdown
Contributor

I got a little hung up yesterday, when trying to use this Action, because it wasn't clear what permissions were required on the PAT that we need to generate to use this project. Making it explicit that you need repo permissions should help folks avoid some guesswork.

@maxheld83
Copy link
Copy Markdown
Owner

yes @alexlafroscia and sorry for the delay! This will be very helpful.

There's a chance this whole manual-PAT-setting-business can be superseded by #18, so I'm going to wait for that for a couple of days.

@alexlafroscia
Copy link
Copy Markdown
Contributor Author

Awesome! That would be even better 😄

@alex-page
Copy link
Copy Markdown
Contributor

I think this can be merged for now. Thoughts @maxheld83?

@maxheld83 maxheld83 merged commit c86d554 into maxheld83:master Sep 25, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants