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 #28: Add github.token as default credential #67

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

Conversation

RammusXu
Copy link
Contributor

No description provided.

@RammusXu RammusXu force-pushed the add-github-token-as-default branch from 838dcba to 9292389 Compare January 28, 2021 04:35
@RammusXu RammusXu force-pushed the add-github-token-as-default branch from 9292389 to 196f8ba Compare January 28, 2021 04:37
@mhausenblas
Copy link
Owner

Thanks @RammusXu and WDYT @janoszen?

@ghost
Copy link

ghost commented Jan 28, 2021

Good idea, but we should still support GITHUB_TOKEN since many have it set up that way.

@mhausenblas
Copy link
Owner

we should still support GITHUB_TOKEN since many have it set up that way.

Thanks and agreed. @RammusXu can you please change, accordingly?

@RammusXu
Copy link
Contributor Author

RammusXu commented Jan 29, 2021

I'm not sure if you want GITHUB_TOKEN > INPUT_AUTH_TOKEN > PERSONAL_TOKEN.

The only way to mess up is using GITHUB_TOKEN: ${{ secrete.PERSONAL_TOKEN }} on this PR, and it won't break existed workflow since default ${{ github.token }} should have enough permission.

Also, I think env.PERSONAL_TOKEN should be the correct way to use for existed users. So I keep it.

@mhausenblas
Copy link
Owner

Thanks for the clarification @RammusXu and I have to admit that I'm not familiar enough with the internals to object "should have enough permission" and "should be the correct way". So, unless I hear @janoszen opposing it, I'll merge ;)

@ghost
Copy link

ghost commented Jan 29, 2021

Currently our documentation states that you can use the GITHUB_TOKEN environment variable to provide a token. Most people will map it to secrets.GITHUB_TOKEN, but that is not mandatory. I would suggest to err on the side of caution and not break the setup people already have.

Alternatively, if we are going to break something let's think of all the design choices we want to change and change it all at once.

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