Skip to content
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

Update docker implementation #1154

Merged
merged 2 commits into from
Feb 1, 2021
Merged

Conversation

egyptianbman
Copy link
Contributor

@egyptianbman egyptianbman commented Feb 1, 2021

I've made various updates to the docker implementation to work with the
latest php7 and currently blocks php8 until we deem nntmux ready. I've
also made modifications that allow for customizations without leaving a
dirty git status i.e. moving redis, etc. to an overrides file that is
gitignored and php customizations i.e. locale to a local overrides file
that is also gitignored.

The new docker implementation utilizes supervisord to run various
services in the web container, i.e. nginx, php-fpm, horizon, etc.

I plan to continue to maintain this as needed. Closes #1152

@egyptianbman egyptianbman force-pushed the docker-update branch 3 times, most recently from 215ff55 to 14a8c47 Compare February 1, 2021 03:46
@egyptianbman egyptianbman mentioned this pull request Feb 1, 2021
I've made various updates to the docker implementation to work with the
latest php7 and currently blocks php8 until we deem nntmux ready. I've
also made modifications that allow for customizations without leaving a
dirty git status i.e. moving redis, etc. to an overrides file that is
gitignored and php customizations i.e. locale to a local overrides file
that is also gitignored.

The new docker implementation utilizes supervisord to run various
services in the web container, i.e. nginx, php-fpm, horizon, etc.

I plan to continue to maintain this as needed. Closes NNTmux#1152
@DariusIII DariusIII self-requested a review February 1, 2021 07:01
@DariusIII DariusIII merged commit 3bdab74 into NNTmux:master Feb 1, 2021
@wimjan123
Copy link

Your awesome man, have great day

@egyptianbman egyptianbman deleted the docker-update branch February 2, 2021 00:12
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.

Docker Outdated
3 participants