Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Improved way of listing URLs in remote#528

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Closed

Conversation

guyzmo
Copy link
Contributor

instead of using git remote show that can trigger a connection to the
remote repository, use git remote get-url --all that works by only
reading the .git/config.

change should have no functional impact, so no test needed.

Signed-off-by: Guyzmoguyzmo+github@m0g.net

expobrainand others added15 commitsSeptember 12, 2016 17:19
Now that performance tests are run, it appears we run into oneparticular failure on travis, possibly indicating a bug in python 3.3.Just bluntly increason the amount of handles might silence it... .Related togitpython-developers#524
instead of using git remote show that can trigger a connection to theremote repository, use git remote get-url --all that works by onlyreading the .git/config.change should have no functional impact, so no test needed.Signed-off-by: Guyzmo <guyzmo+github@m0g.net>
@guyzmo
Copy link
ContributorAuthor

(I noticed the issue when doing a.urls while on the train without internet connection 😖)

@ankostis
Copy link
Contributor

ankostis commentedOct 11, 2016
edited
Loading

@guyzmoLinuxWindows checks fail due#513 BUTLinux checks also fail due togit.test.test_remote:TestRemote.test_multiple_urls () failing due to (I suppose) TravisCI having an older version of git; check my own build where I don't merge with the buggy master branch:https://travis-ci.org/ankostis/GitPython/jobs/166806778

ankostis pushed a commit to ankostis/GitPython that referenced this pull requestOct 11, 2016
+ Instead of using `git remote show` that may triggers connection toremote repo, use `git remote get-url --all` that works by onlyreading the `.git/config`.+ Change should have no functional impact, so no test needed.+ Works only with git -2.7+.Signed-off-by: Guyzmo <guyzmo+github@m0g.net>
ankostis added a commit to ankostis/GitPython that referenced this pull requestOct 11, 2016
ankostis pushed a commit to ankostis/GitPython that referenced this pull requestOct 12, 2016
@guyzmo
Copy link
ContributorAuthor

looks like it's merged, thanks 👍

@guyzmoguyzmo closed thisOct 13, 2016
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

4 participants
@guyzmo@ankostis@expobrain@Byron

[8]ページ先頭

©2009-2025 Movatter.jp