Skip to content

Conversation

letusgitcommit
Copy link

Describe what this PR does / why we need it

This PR adds a note to the README concerning possibly running into authentication errors that I experienced not fully understanding the need to add ssh aliases to the setup config.

Does this pull request fix one issue?

I didn't create an issue 🤷‍♂️. It felt easier just to update the README since I wasn't making any code changes. If you need me to, just let me know.

"NONE"

Describe how you did it

I forked the repo, cloned locally, checked out my branch, and typed in the addition.

Describe how to verify it

The creator should verify whether this is correct.

Special notes for reviews

Thanks for creating this project. Haven't gotten to fully use it yet, but it looks awesome.

Didn't change any code so I did not run any tests or linters.

Checklist

  • Passing tests and linting standards
  • Documentation updates in README.md and doc/octo.txt

README.md Outdated
@@ -691,6 +691,15 @@ GITHUB_TOKEN= gh auth login

`gh` must store the credentials so it can work in a subshell.

ALSO:
Copy link
Collaborator

Choose a reason for hiding this comment

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

Can you use a markdown block with an appropriate section label

Copy link
Author

@letusgitcommit letusgitcommit Sep 12, 2025

Choose a reason for hiding this comment

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

@williambdean
Please review. I meant to include as part of that section previous (thus the part 2 haha). If there is something specific that you're looking for, please let me know.

Please resolve at your leisure.

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