Skip to content

Commit

Permalink
add docker cli as a requirement during check (#454)
Browse files Browse the repository at this point in the history
Docker does not officially provide LTS versions, picked 18.0.0 as a
baseline because it seems like a reasonable timeframe(2018), appears
they are already out of support cycle. I think we should encourage
but not enforce using newer versions that receives security updates.
  • Loading branch information
davidcheung committed Jan 28, 2022
1 parent a7afa04 commit 75d3c9f
Showing 1 changed file with 8 additions and 0 deletions.
8 changes: 8 additions & 0 deletions cmd/check.go
Original file line number Diff line number Diff line change
Expand Up @@ -119,6 +119,14 @@ var checkCmd = &cobra.Command{
minVersion: "1.15.2",
docsURL: "https://kubernetes.io/docs/tasks/tools/install-kubectl/",
},
{
name: "Docker\t\t",
command: "docker",
args: []string{"--version"},
regexStr: `Docker version (0|[1-9]\d*)\.(0|[1-9]\d*)\.(0|[1-9]\d*), build ([a-f0-9]{7})`,
minVersion: "18.0.0",
docsURL: "https://docs.docker.com/get-docker/",
},
{
name: "Terraform\t",
command: "terraform",
Expand Down

0 comments on commit 75d3c9f

Please sign in to comment.