-
Notifications
You must be signed in to change notification settings - Fork 18
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
Check for docker-compose on init #65
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Very often (basically always), I have a separate compose yaml for prod (whenever I do this type of deployment technique). I think this is OK for now, but it might make sense to add a flag in the future.
inertia init --compose-file docker-compose-prod.yml
assert.Equal(t, nil, CheckForGit(inertia)) | ||
} | ||
|
||
func TestCheckForDockerCompose(t *testing.T) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We need containers for this type of testing π
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe, but a container would probably be a bit overkill for this because all it does is check for the presence of the docker-compose.yml
file
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Its more this contamination of the developers file system during tests. Say someone creates a docker-compose.yml
file in this repo one day, then they run this test. The test failing is the best we can hope for, the worst is that it nukes their file.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ah spooky
Most will stil have the standard |
β Status: ready
ποΈ Ticket(s): Close #36 Warn if
inertia init
in non-docker-compose projectπ· Changes
inertia init
π¦ Testing Instructions