Skip to content
This repository has been archived by the owner on Nov 19, 2024. It is now read-only.

Add docker-compose support #6890

Merged
merged 9 commits into from
Mar 20, 2020
Merged

Conversation

cmuench
Copy link
Contributor

@cmuench cmuench commented Mar 19, 2020

Purpose of this pull request

This pull request (PR) adds support for Docker to quickly setup the project. This speeds up onbording of new devdocs contributors.

Added the ability to build and preview DevDocs in a Docker container. See the DevDocs Readme for directions.

@devops-devdocs
Copy link
Collaborator

An admin must run tests on this PR before it can be merged.

@hguthrie
Copy link
Contributor

This builds properly for me.

@hguthrie hguthrie requested review from jcalcaben and hguthrie March 19, 2020 16:37
@@ -141,6 +142,20 @@ exclude:
>rake preview:all
>```

## Docker (docker-compose)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

To add some context to this feature, can you add the following information in a summary before the instructions:

  1. What are the use cases for using this feature?
  2. Do users need anything else besides docker installed?
  3. Will this work from a fresh copy of the repo or do people need to perform some pre-requisite steps?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's quite simple. Only a docker installation is needed. Should work in every repo/fork. I can add an additional comment.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@jcalcaben I added an description. Please review it. I am not a native speaker.

README.md Outdated Show resolved Hide resolved
Copy link
Contributor

@hguthrie hguthrie left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This may help some with some clarity. And I will try to see it reloads for me...

README.md Outdated Show resolved Hide resolved
README.md Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
cmuench and others added 3 commits March 19, 2020 20:10
Co-Authored-By: hguthrie <hguthrie@users.noreply.github.com>
## Docker (docker-compose)

Docker provides a quick and easy way to build and preview the documentation. You do not need to install Ruby dependencies manually. This Docker solution launches Jekyll inside the container in `watch` mode.```
ruby dependency manually. The provided setup starts Jekyll inside the container in watch mode.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You don't need this line now, since the updates were made. 🙂

README.md Outdated Show resolved Hide resolved
README.md Show resolved Hide resolved
Copy link
Contributor

@hguthrie hguthrie left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks! 🏅

@hguthrie hguthrie added the Technical Updates to the code or processes that alter the technical content of the doc label Mar 19, 2020
@hguthrie
Copy link
Contributor

running tests

@hguthrie hguthrie merged commit a7df034 into magento:master Mar 20, 2020
@ghost
Copy link

ghost commented Mar 20, 2020

Hi @cmuench, thank you for your contribution!
Please, complete Contribution Survey, it will take less than a minute.
Your feedback will help us to improve contribution process.

@dshevtsov dshevtsov removed the Technical Updates to the code or processes that alter the technical content of the doc label Mar 23, 2020
@dshevtsov
Copy link
Collaborator

The Technical label was removed because it can be applied only to changes in content, not infrastructure.

@jeff-matthews jeff-matthews added the Technical Updates to the code or processes that alter the technical content of the doc label Mar 23, 2020
@jeff-matthews
Copy link
Contributor

jeff-matthews commented Mar 23, 2020

Adding the label back in recognition of the contributor's efforts. It's not a common use case, so I'm allowing an exception this time.

@cmuench
Copy link
Contributor Author

cmuench commented Mar 24, 2020

@jeff-matthews thanks. Thought... a simple Docker setup can help to speed up the onboarding process. So more people can contribute.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Partner: Netz98 partners-contribution PR created by Magento partner Technical Updates to the code or processes that alter the technical content of the doc
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants