Skip to content

Conversation

@mrnugget
Copy link
Contributor

We had the check for git before, but lost it in the switch to the new
model.

The Docker check I added after running into a hard-to-decipher error
when trying to run src campaigns apply on a VM without Docker.

Here's what it looks like:

screenshot_2020-09-22_16 37 30@2x

@mrnugget mrnugget requested a review from a team September 22, 2020 14:39
We had the check for git before, but lost it in the switch to the new
model.

The Docker check I added after running into a hard-to-decipher error
when trying to run `src campaigns apply` on a VM without Docker.
@mrnugget mrnugget force-pushed the mrn/check-executables branch from bd835f4 to 2b91492 Compare September 23, 2020 09:08
@mrnugget mrnugget merged commit 1959e8d into main Sep 23, 2020
@mrnugget mrnugget deleted the mrn/check-executables branch September 23, 2020 09:27
scjohns pushed a commit that referenced this pull request Apr 24, 2023
…326)

* Check that Docker and git are available when executing campaign spec

We had the check for git before, but lost it in the switch to the new
model.

The Docker check I added after running into a hard-to-decipher error
when trying to run `src campaigns apply` on a VM without Docker.

* Add changelog entry
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