Pete Halverson

ENVIRONMENT required. Needs fallback

2

The `ENVIRONMENT` constant you're referencing ( https://github.com/agungjk/phpdotenv-for-codeigniter/blob/master/system/dotenv/Dotenv.php#L29) may not be defined. I think Dotenv should fall back to a standard `.env` if that's the case.

Revisit composer install triggered by `INIT_PLUGINS` env var

5

The [current method](https://github.com/aspendigital/docker-octobercms/blob/master/docker-oc-entrypoint#L109-L121) may cause conflicting dependencies.

When using `GIT_CHECKOUT` or `GIT_MERGE_PR` it's possible composer libraries have been updated. Warn the user if that's the case. Consider prompting user to run composer

Permission issues introduced when running artisan tasks as root

2

Permissions on files changed by running command line tasks has the potential to reset permissions as root:root (e.g. file based cache)

@shell multi-line commands

2

Multi-line shell commands don't seem to be supported as the Wiki example indicates.