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

Ensure the PAT option is always offered for GHES instances #259

Merged
merged 1 commit into from
Jan 6, 2021

Conversation

mjcheetham
Copy link
Collaborator

Ensure the Personal Access Token authentication mode is always offered for GitHub Enterprise Server instances.

Ensure the Personal Access Token authentication mode is always offered
for GitHub Enterprise Server instances.
@mjcheetham mjcheetham added bug A bug in Git Credential Manager host:github Specific to the GitHub host provider labels Jan 6, 2021
Copy link
Contributor

@derrickstolee derrickstolee left a comment

Choose a reason for hiding this comment

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

I see, you essentially add Pat to all possible results. Sure.

@mjcheetham
Copy link
Collaborator Author

I see, you essentially add Pat to all possible results. Sure.

Yep. I missed this when another PR that changed how we conduct the unit tests around this came in ahead, and I had to rebase/resolve. Missed the PAT vs None case.

@mjcheetham mjcheetham merged commit 8f96623 into git-ecosystem:master Jan 6, 2021
@mjcheetham mjcheetham deleted the fix-ghes-modes branch January 6, 2021 16:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug A bug in Git Credential Manager host:github Specific to the GitHub host provider
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants