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

Remove remote prefix from upload printout #111

Merged
merged 1 commit into from
Apr 13, 2023
Merged

Conversation

jerry-skydio
Copy link
Collaborator

When using a fork this prefix is confusing since the
PR is pushed to origin but the github repo used is "fork".
When not using a fork there is only one remote so it isn't
useful. To simplify both cases, don't print the prefix.

Topic: nopre
Reviewers: brian-k, aaron-m

When using a fork this prefix is confusing since the
PR is pushed to origin but the github repo used is "fork".
When not using a fork there is only one remote so it isn't
useful. To simplify both cases, don't print the prefix.

Topic: nopre
Reviewers: brian-k, aaron-m
@jerry-skydio
Copy link
Collaborator Author

Reviews in this chain:
#111 Remove remote prefix from upload printout

@jerry-skydio
Copy link
Collaborator Author

# head base diff date summary
0 3f260748 caa2b08d diff Apr 13 11:15 AM 1 file changed, 3 insertions(+), 2 deletions(-)

@jerry-skydio jerry-skydio merged commit ab93ff9 into main Apr 13, 2023
@jerry-skydio jerry-skydio deleted the jerry/revup/main/nopre branch April 13, 2023 21:04
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