Skip to content

Conversation

@pabelanger
Copy link
Contributor

We don't properly encode our error message under python3.

Signed-off-by: Paul Belanger [email protected]

We don't properly encode our error message under python3. Signed-off-by: Paul Belanger <[email protected]>
@codecov-io
Copy link

Codecov Report

Merging #682 into master will not change coverage.
The diff coverage is 0%.

Impacted file tree graph

@@ Coverage Diff @@## master #682 +/- ## ======================================= Coverage 92.95% 92.95% ======================================= Files 61 61 Lines 10065 10065 ======================================= Hits 9356 9356 Misses 709 709
Impacted FilesCoverage Δ
git/cmd.py83.4% <0%> (ø)⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 95ff827...1dbfd29. Read the comment docs.

@Byron
Copy link
Member

Byron commented Oct 7, 2017

Thanks a lot for your contribution!

@ByronByron merged commit f237620 into gitpython-developers:masterOct 7, 2017
openstack-gerrit pushed a commit to openstack-infra/zuul that referenced this pull request Oct 10, 2017
Timeout remote git operations after 300 seconds. Because it could be in an invalid state, delete the local repo if a timeout occurs (subsequent operations will recreate it). This replaces our use of the clone_from() and fetch() methods from GitPython with lower-level equivalents. The high-level methods do not currently permit the hard timeout. The GitPython requirement is changed to a temporary fork until both gitpython-developers/GitPython#682 and gitpython-developers/GitPython#686 end up in a release. Change-Id: I7f680472a8d67ff2dbe7956a8585fb3714119e65
riley-martine pushed a commit to riley-martine/GitPython that referenced this pull request Dec 7, 2023
…ding Fix encoding issue with stderr_value and kill_after_timeout
Sign up for freeto 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.

3 participants

@pabelanger@codecov-io@Byron