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

Apply updates for GitHub AE. #51

Merged
merged 1 commit into from
Jan 20, 2021
Merged

Apply updates for GitHub AE. #51

merged 1 commit into from
Jan 20, 2021

Conversation

chrisgavin
Copy link
Collaborator

This applies the changes needed to make the sync tool work with GitHub AE. The primary addition is that we now detect when we are syncing to GitHub AE and instead of making a public repository we instead make an internal repository. This has also required a few changes to how we handle scopes as internal repositories cannot be accessed by tokens with the public_repo scope.

@simon-engledew simon-engledew self-requested a review January 19, 2021 12:07
Copy link
Contributor

@simon-engledew simon-engledew left a comment

Choose a reason for hiding this comment

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

I have tested this against GHAE and GHES and it works great!

Really clear and helpful error messages. ❤️

@chrisgavin chrisgavin merged commit a1bc449 into main Jan 20, 2021
@chrisgavin chrisgavin deleted the aegis branch January 20, 2021 12:33
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