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

Allow spr format to be used without GitHub credentials #113

Conversation

sven-of-cord
Copy link
Contributor

The format command does not use the GitHub API, it only reformats the local commit message.
This commit makes it so that spr format can be called without a GitHub PAT in the config.
This is particularly useful to have a slightly more meaningful automated test of the spr build. For example homebrew allows to define tests of the binary as part of the packaging recipe. With this change, we can test using spr format, which at least tests that a Git repository is opened correctly.

Test Plan:
use spr format and see that it correctly reformats a messy commit message

Created using spr 1.3.4-beta.1
Copy link
Contributor

@jozef-mokry jozef-mokry left a comment

Choose a reason for hiding this comment

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

lgtm

Since spr format does not use the token spr format --github-auth-token none also worked as a workound

@sven-of-cord
Copy link
Contributor Author

lgtm

Since spr format does not use the token spr format --github-auth-token none also worked as a workound

Oh wow, yes, true. Hadn't thought of that. Ah well, might as well land this now.

@sven-of-cord sven-of-cord merged commit 5b62aee into master Jul 18, 2022
@sven-of-cord sven-of-cord deleted the spr/sven-of-cord/allow-spr-format-to-be-used-without-github-credentials branch July 18, 2022 13:06
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.

3 participants