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

Evaluate Docker Compose for local development environment instead of shell script #36

Closed
jwflory opened this issue Dec 2, 2018 · 0 comments · Fixed by #88
Closed

Evaluate Docker Compose for local development environment instead of shell script #36

jwflory opened this issue Dec 2, 2018 · 0 comments · Fixed by #88
Assignees
Labels
good first issue Good for newcomers help wanted Extra attention is needed improvement Improves on something that already exists

Comments

@jwflory
Copy link
Member

jwflory commented Dec 2, 2018

@Nolski mentioned idea of using Docker Compose instead of the shell script introduced in #35. It would be a cleaner way of managing the development environment. Given my previous experience with it in other projects, I think it would be a nice change.

I'm filing this for consideration in a post-v1.0 pull request. If someone has time and wants to play with this, feel free to try taking this one on!

@jwflory jwflory added improvement Improves on something that already exists help wanted Extra attention is needed labels Dec 2, 2018
@jwflory jwflory added the good first issue Good for newcomers label Dec 2, 2018
@ghost ghost changed the title Evalaute Docker Compose for local development environment instead of shell script Evaluate Docker Compose for local development environment instead of shell script Nov 5, 2019
@jwflory jwflory self-assigned this Feb 17, 2020
jwflory added a commit that referenced this issue Feb 17, 2020
This commit drops the old `start_devel_env.sh` script in favor of a more
versatile `build.sh` script. This takes inspiration from tooling written
for the Fedora Docs project in using different container runtimes based
on what the user has installed to build a site (e.g. Docker or Podman).

I considered using Docker Compose for this, but I think the future of
rootless containers makes Docker Compose less interesting of a tool
choice, because it is locked into the Docker container runtime. This
script still uses Docker on macOS and Linux, but it also provides the
possibility to use other container runtimes like Podman on Linux. Thus,
I think this reworking of the dev environment build script is enough to
close out issue #36.

Note this commit does not update the docs, but this content will be
moved to the Runbook instead of this repo. This is being tracked in
#86.

Closes #36.

Signed-off-by: Justin W. Flory <git@jwf.io>
jwflory added a commit that referenced this issue Feb 20, 2020
This commit drops the old `start_devel_env.sh` script in favor of a more
versatile `build.sh` script. This takes inspiration from tooling written
for the Fedora Docs project in using different container runtimes based
on what the user has installed to build a site (e.g. Docker or Podman).

I considered using Docker Compose for this, but I think the future of
rootless containers makes Docker Compose less interesting of a tool
choice, because it is locked into the Docker container runtime. This
script still uses Docker on macOS and Linux, but it also provides the
possibility to use other container runtimes like Podman on Linux. Thus,
I think this reworking of the dev environment build script is enough to
close out issue #36.

Note this commit does not update the docs, but this content will be
moved to the Runbook instead of this repo. This is being tracked in
#86.

Closes #36.

Signed-off-by: Justin W. Flory <git@jwf.io>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers help wanted Extra attention is needed improvement Improves on something that already exists
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant