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

git: Don't swallow all push output #203

Closed
wants to merge 1 commit into from

Conversation

jerry-skydio
Copy link
Collaborator

In certain situations "git push" will need to auth interactively. However
we don't show the user any output from git push because of github's automatic
"create a pr test" that is generally not what we want since revup handles pr
creation. This does make these auth situations confusing though.

Instead of hiding all output, default to showing output except for lines prefixed
with "remote:" which will get hidden.

Topic: outpush2
Closes: #33

In certain situations "git push" will need to auth interactively. However
we don't show the user any output from git push because of github's automatic
"create a pr test" that is generally not what we want since revup handles pr
creation. This does make these auth situations confusing though.

Instead of hiding all output, default to showing output except for lines prefixed
with "remote:" which will get hidden.

Topic: outpush2
Closes: #33
@jerry-skydio
Copy link
Collaborator Author

Reviews in this chain:
#203 git: Don't swallow all push output

@jerry-skydio
Copy link
Collaborator Author

# head base diff date summary
0 b12adb1e c7b92e83 diff Dec 28 16:22 PM 2 files changed, 20 insertions(+), 6 deletions(-)

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.

Password protected keys unsupported
1 participant