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

Pull in fork test from hub #190

Closed
wants to merge 1 commit into from
Closed

Pull in fork test from hub #190

wants to merge 1 commit into from

Conversation

owenthereal
Copy link
Owner

@owenthereal
Copy link
Owner Author

@mislav I'm trying to pull in some compatibility code from hub. But I'm not sure how the changes fix the asynchronous forking repo availability issue: it first remote adds and then set-url. Is there any magic I missed from git 😸?

@mislav
Copy link
Collaborator

mislav commented Jan 30, 2014

Before, it would add the "jingweno" remote and immediately try to fetch from your fork. In some cases, the new git repo wouldn't be ready yet on GitHub.com.

Now, it adds the "jingweno" remote to point to "origin", fetches that, then changes the URL to actually point to your fork.

@owenthereal
Copy link
Owner Author

Ah, I think I missed that git remote add -f runs git fetch as well. Thanks @mislav

@owenthereal
Copy link
Owner Author

Actually why do we need to fetch? Isn't that the root cause of the problem is fetching a repo that is potentially not ready? How about just git remote add without the fetch and let the users fetch themselves?

@mislav
Copy link
Collaborator

mislav commented Jan 31, 2014

Yes, the root cause was the fetch, but now we're fetching from origin, which is sure to be ready ;) and renaming that to "yourfork" after the fetch.

The fetch isn't crucial here, but I'd like to have the current state of user's fork reflected in the remote tracking branches so that future operations are predictable, like checking if you've pushed all commits before making a pull-request, and similar.

@owenthereal
Copy link
Owner Author

Closing. Further development goes to mislav/hub#475

@owenthereal owenthereal closed this Feb 7, 2014
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