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

Recognize multiple remotes #67

Merged
merged 3 commits into from
Mar 18, 2017
Merged

Recognize multiple remotes #67

merged 3 commits into from
Mar 18, 2017

Conversation

himynameisdave
Copy link
Owner

A common open source workflow includes having an origin remote for my fork and an upstream fork for the primary project. I found, to use this tool, I have to rename my upstream to origin for it to work.

I'm good with either updating documentation to include that note or adding additional logic to ask which remote is the right one. Your call 👍

intelsdi-x/snap # git remote rename origin upstream
intelsdi-x/snap # git remote
mjbrender
upstream
intelsdi-x/snap # git-labelmaker
? You have a saved token.
Would you like to unlock & use this token, or create a new one? Use Saved Token
? What is your master password?
TypeError: Cannot read property 'url' of undefined
    at module.exports (/usr/local/lib/node_modules/git-labelmaker/bin/modules/readRepo.js:13:40)
    at Promise.all.then (/usr/local/lib/node_modules/git-labelmaker/bin/index.js:38:19)
 ✘ intelsdi-x/snap # git remote rename upstream origin
intelsdi-x/snap # git-labelmaker
? You have a saved token.
Would you like to unlock & use this token, or create a new one? Use Saved Token
? What is your master password?
=======================================
]|[                                 ]|[
]|[    Welcome to git-labelmaker    ]|[
]|[                                 ]|[
=======================================

? Welcome to git-labelmaker!
What would you like to do? Quit
=======================================
]|[                                 ]|[
]|[            See Ya!              ]|[
]|[                                 ]|[
=======================================

There is some stuff that needs to be cleaned up though for sure
@himynameisdave himynameisdave merged commit c539db5 into v0.9.0 Mar 18, 2017
@himynameisdave himynameisdave deleted the issues/67 branch March 18, 2017 21:01
@himynameisdave
Copy link
Owner

@mbbroberg sorry it's taking so long, but as of the 0.9.0 release (which should be ready for publishing soon) you will be prompted for which remote url you want to use.

Thanks for this suggestion 👍

giphub

@mbbroberg
Copy link
Author

Great update, thanks so much!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants