Skip to content

Update GitRepo to use pygit2#8

Merged
white-gecko merged 3 commits intomasterfrom
feature/pygit2
Nov 29, 2016
Merged

Update GitRepo to use pygit2#8
white-gecko merged 3 commits intomasterfrom
feature/pygit2

Conversation

@splattater
Copy link
Contributor

This commit changes the git library from GitPython to pygit2.
With this commit the installation of QuitStore changes and libgit2 is needed to fulfill the requirements of pygit2.

This commit changes the git library from GitPython to pygit2.
With this commit the installation of QuitStore changes and libgit2 is needed to fulfill the requirements of pygit2.
@coveralls
Copy link

coveralls commented Nov 24, 2016

Coverage Status

Coverage increased (+0.1%) to 13.391% when pulling 0fcec64 on feature/pygit2 into 35e08b7 on master.

@white-gecko
Copy link
Member

The quit.log is also commited and thus my verification script fails.

@coveralls
Copy link

coveralls commented Nov 25, 2016

Coverage Status

Coverage increased (+0.2%) to 13.46% when pulling a085742 on feature/pygit2 into 35e08b7 on master.

@splattater
Copy link
Contributor Author

I added an option to enable "pathspec" in pygit2 when using the index_add_all method.
The method will be called with a list containing all files that are configured in config.ttl, so every unknown/untracked files won't be added to index.
You could also update your gitignore to avoid that issue.

@white-gecko
Copy link
Member

With --pathspec option it creates empty commits.

Since changing GitRepo method from "git add -u" to pygit2, it is necessary to
check if an update query will change store content.
If not, we won't commit.
@coveralls
Copy link

coveralls commented Nov 28, 2016

Coverage Status

Coverage increased (+0.2%) to 13.448% when pulling 097f2f0 on feature/pygit2 into 35e08b7 on master.

@white-gecko
Copy link
Member

@white-gecko white-gecko merged commit 60fe239 into master Nov 29, 2016
@white-gecko white-gecko deleted the feature/pygit2 branch November 29, 2016 10:17
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.

3 participants