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

Eat stderr messages from git commands. #399

Merged
merged 2 commits into from Aug 22, 2022
Merged

Eat stderr messages from git commands. #399

merged 2 commits into from Aug 22, 2022

Conversation

tarmack
Copy link
Contributor

@tarmack tarmack commented Jul 26, 2022

Prevents "fatal: not a git repository (or any of the parent directories): .git" in cases where safety is ran outside a git repository.

Prevents:
"fatal: not a git repository (or any of the parent directories): .git"
in cases where safety is ran outside a git repository.
@codecov
Copy link

codecov bot commented Aug 18, 2022

Codecov Report

Merging #399 (e010baa) into develop (00d4633) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff            @@
##           develop     #399   +/-   ##
========================================
  Coverage    86.05%   86.06%           
========================================
  Files           14       14           
  Lines         1714     1715    +1     
========================================
+ Hits          1475     1476    +1     
  Misses         239      239           
Impacted Files Coverage Δ
safety/util.py 81.41% <100.00%> (+0.04%) ⬆️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

Copy link
Member

@yeisonvargasf yeisonvargasf left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@tarmack, thanks for this contribution!

@yeisonvargasf yeisonvargasf merged commit 7895e10 into pyupio:develop Aug 22, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants