Skip to content

try/except block to handle use-cases without git#40

Open
mathematicalmichael wants to merge 2 commits intoabraham-ai:masterfrom
mathematicalmichael:patch-1
Open

try/except block to handle use-cases without git#40
mathematicalmichael wants to merge 2 commits intoabraham-ai:masterfrom
mathematicalmichael:patch-1

Conversation

@mathematicalmichael
Copy link

@mathematicalmichael mathematicalmichael commented Mar 12, 2023

ran into an issue trying to build a container (which did not depend on git cloning an "eden app"), and currently the code will bail out on git.Repo. This proposed change allows for overriding the need for git to be installed by setting an environment variable, and allows for successful server responses without it.

ran into an issue trying to build a container which does not depend on git cloning an "eden app", and currently the code will bail out on `git.Repo`.
@codecov-commenter
Copy link

Codecov Report

Merging #40 (fca926b) into master (04b6b3d) will decrease coverage by 0.56%.
The diff coverage is n/a.

📣 This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more

@@            Coverage Diff             @@
##           master      #40      +/-   ##
==========================================
- Coverage   69.04%   68.49%   -0.56%     
==========================================
  Files          22       24       +2     
  Lines         588      657      +69     
==========================================
+ Hits          406      450      +44     
- Misses        182      207      +25     

see 2 files with indirect coverage changes

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

`bare` is not an arg to this function...
@mathematicalmichael
Copy link
Author

I tested this change with the same COPY directive as in #41 and can confirm that this change allows a container to be built without git installed if ENV GIT_PYTHON_REFRESH=quiet is set in the Dockerfile.

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.

2 participants