FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

import OrderedDict from git.odict rather than directly from collections, to pix Py2.6 compatibility by boegel · Pull Request #443 · gitpython-developers/GitPython · GitHub

Repository navigation

import OrderedDict from git.odict rather than directly from collections, to pix Py2.6 compatibility - #443

Merged
Byron merged 1 commit into
gitpython-developers:masterfrom
boegel:fix_ordereddict
May 26, 2016
Merged

Byron merged 1 commit into
gitpython-developers:masterfrom
boegel:fix_ordereddict

Conversation

boegel commented May 26, 2016

Copy link
Copy Markdown
Contributor

cfr. https://github.com/gitpython-developers/GitPython/pull/431/files/89ade7bfff534ae799d7dd693b206931d5ed3d4f#r64649040

cc @Byron

I tested this in my fork on Travis; I'm running into one failing test, but it seems unrelated to this change.
Am I missing something to make the tests pass?

======================================================================
ERROR: test_init_repo_object (git.test.test_docs.Tutorials)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/home/travis/build/boegel/GitPython/git/ext/gitdb/gitdb/test/lib.py", line 87, in wrapper
    return func(self, path)
  File "/home/travis/build/boegel/GitPython/git/test/test_docs.py", line 68, in test_init_repo_object
    assert repo.refs['origin/master'] == repo.remotes.origin.refs.master  # ... remotes ...
  File "/home/travis/build/boegel/GitPython/git/util.py", line 684, in __getitem__
    raise IndexError("No item found with id %r" % (self._prefix + index))
IndexError: No item found with id 'origin/master'

Byron added this to the v2.0.4 - Bugfixes milestone May 26, 2016
Byron self-assigned this May 26, 2016

Byron commented May 26, 2016

Copy link
Copy Markdown
Member

The unit-tests are not perfectly isolated, and being on a branch makes some of them fail. It's a known issue, also one that I am not proud of :).
Travis is happy, here you go, thanks for your contribution !

Byron merged commit 8486f2d into gitpython-developers:master May 26, 2016

boegel commented May 26, 2016

Copy link
Copy Markdown
Contributor Author

Thanks @Byron!

This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants


Back | FazBrowse Home | New Git URL