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.

Sign up for GitHub

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

Jump to bottom

Clarify GitHub token behaviour #53

Merged
merged 1 commit into from
May 23, 2023
Merged

Clarify GitHub token behaviour #53

merged 1 commit into from
May 23, 2023

Conversation

jcbhmr
Copy link
Collaborator

@jcbhmr jcbhmr commented May 19, 2023 •
edited
Loading

closes #52

@jcbhmr jcbhmr added the documentation Improvements or additions to documentation label May 19, 2023
@jcbhmr jcbhmr requested a review from Andrew-Chen-Wang May 19, 2023 22:38
@jcbhmr jcbhmr self-assigned this May 19, 2023
@jcbhmr jcbhmr marked this pull request as draft May 19, 2023 22:38
@jcbhmr
Copy link
Collaborator Author

jcbhmr commented May 19, 2023 •
edited
Loading

I usually refer to github.token as "Auto-included GitHub Action token"

I meant a distinction between when to use this action token and when to use a PAT.

#52 (comment)

@Andrew-Chen-Wang Andrew-Chen-Wang marked this pull request as ready for review May 20, 2023 06:06
@Andrew-Chen-Wang
Copy link
Owner

lgtm! thanks again!

@jcbhmr jcbhmr merged commit 2a6935c into master May 23, 2023
@jcbhmr jcbhmr deleted the clarify-token branch May 23, 2023 00:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Andrew-Chen-Wang Andrew-Chen-Wang Awaiting requested review from Andrew-Chen-Wang

Assignees

@jcbhmr jcbhmr

Labels
documentation Improvements or additions to documentation
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Clarify GitHub auto-token vs GitHub PAT
2 participants
@jcbhmr @Andrew-Chen-Wang