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

Document GITHUB_TOKEN workaround #1239

Closed
wants to merge 3 commits into from
Closed

Document GITHUB_TOKEN workaround #1239

wants to merge 3 commits into from

Conversation

bramtayl
Copy link

Closes #1177

for more information.
While GitHub will automatically create a github token for you named "GITHUB_TOKEN",
it will not have push access. Therefore, you must override
GitHub by making a new personal access token with the same name.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It should specifically say what to do.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I can add a note like (see instructions above) because I did add instructions to do this?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, and a link to that part of the instructions

@aminya
Copy link
Contributor

aminya commented Feb 16, 2020

I also think more adjustment to the documentation is needed based on what I said here:
#1177 (comment)

@bramtayl
Copy link
Author

Now that github actions is up and running maybe we should just ditch all the travis and ssh-key docs?

@aminya
Copy link
Contributor

aminya commented Feb 16, 2020

Now that github actions is up and running maybe we should just ditch all the travis and ssh-key docs?

I agree. Probably it is better to move them to a separate document called Travis.md.

@fredrikekre
Copy link
Member

fredrikekre commented Feb 16, 2020

IMO we should not Document this option and only recommend SSH keys (as we already do).

@bramtayl
Copy link
Author

Why?

@fredrikekre
Copy link
Member

Because access tokens are not limited to one repo, and is tied to a user, so it would be like giving your github password to everyone with write access to the repo in question.

@bramtayl
Copy link
Author

Hmm I think I see what your saying. Still seems like a pain to make SSH keys for each individual repo. I wonder if there's another solution?

@fredrikekre
Copy link
Member

You can reuse it if you want

@mortenpi
Copy link
Member

I have to agree with @fredrikekre here: we shouldn't recommend the use of personal access tokens for this.

You can reuse [the SSH deploy key] if you want

I don't think this is true. If you try to add the same deploy key again, GitHub will complain if I remember correctly.

Now that github actions is up and running maybe we should just ditch all the travis and ssh-key docs?

As long as we don't have #1177, we can't ditch the deploy key stuff. We could have GitHub Actions as the recommended setup (even as we're waiting for #1177), but no reason to ditch Travis -- it can still be useful for some people.

@bramtayl
Copy link
Author

Close in favor of #1244

@bramtayl bramtayl closed this Feb 19, 2020
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.

GitHub Pages build not triggered with GITHUB_TOKEN
4 participants