apogiatzis / docker-compose-livereloader

A docker compose pattern to enable automatic container reloading.
77 stars 12 forks source link

Add option to restart exited containers #11

Closed Emiliaaah closed 2 years ago

Emiliaaah commented 3 years ago

Description

This environment variable let's the user choose to also restart containers exited containers.

Where is this useful?

This option is useful for users that don't want to have to restart their container manually if they made a fatal code change that caused their container to exit.

Emiliaaah commented 2 years ago

@apogiatzis Just curious if you could merge my PR and if I have to make any changes.

apogiatzis commented 2 years ago

Hey @Emiliaaah thanks for your contribution and apologies for the late response! Your feature suggestion makes sense so I am merging your PR now! Thanks!

Emiliaaah commented 2 years ago

No problem, I looked on docker hub but there doesn't seem to be a new build yet, do you know when can I expect one?

apogiatzis commented 2 years ago

No problem, I looked on docker hub but there doesn't seem to be a new build yet, do you know when can I expect one?

Hey @Emiliaaah! I haven't setup any GitHub actions for automatic push to DockerHub, the one you in DockerHub is an old built that I used personally some time ago that's why it's one my dockerhub profile.

For using this I would suggest cloning the repo, build it and either use the locally built image or upload it on a personal/business image registry. This is more preferable apprach from a cyber security perspective.