Skip to content
This repository has been archived by the owner on Jan 10, 2023. It is now read-only.

Warn if git is missing #84

Closed
lazka opened this issue Aug 10, 2017 · 4 comments
Closed

Warn if git is missing #84

lazka opened this issue Aug 10, 2017 · 4 comments
Assignees

Comments

@lazka
Copy link

lazka commented Aug 10, 2017

Using the bash script in docker fails to get the config because by default most docker images don't have git installed.

I figured it out eventually, but a warning/error would have been nice.

@stevepeak
Copy link
Contributor

@lazka We have another technique where you export the report(s) out of docker and upload them with git as a command. See here: https://docs.codecov.io/docs/testing-with-docker

@Midnighter
Copy link

Still, on that very page a warning that the technique inside requires git would have saved us quite a bit of trouble.

@rgov
Copy link

rgov commented Feb 4, 2019

I was going to file the same bug for curl. Is there a list of all the tools that the script relies on that are not included in, say, a base Debian image? Can these all be tested for by the script?

@thomasrockhu
Copy link
Contributor

Fixed via #394

Change Tracking automation moved this from To do to Done Jan 3, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
Development

Successfully merging a pull request may close this issue.

5 participants