Skip to content
This repository was archived by the owner on Dec 3, 2024. It is now read-only.

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 subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants