<p>Hey guys lot's of comments. I can't address them all right at the moment but to pick one:</p>

<blockquote>
<p>There should be a COPY or ADD instruction to get the local directory to the docker instance running the rails port. I'm not sure on the semantics of copy vs add, so I don't know which is preferred.</p>
</blockquote>

<p>This isn't required as the <code>docker-compose.yml</code> specifies <code>/app</code> as the <code>volume</code> which exposes the code locally on your machine to make it available in the container.</p>

<p>This means you can make changes locally and see them reflected when you refresh the browser.</p>

<p>This would obviously be different you wanted to use Docker to deploy to a production server where it would need to clone down the repo. Hopefully that makes sense!</p>

<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">—<br />You are receiving this because you are subscribed to this thread.<br />Reply to this email directly, <a href="https://github.com/openstreetmap/openstreetmap-website/pull/1290#issuecomment-247539093">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/ABWnLSRnczWdOGqiX8D0Hso0u204yDjyks5qqklcgaJpZM4J-dnj">mute the thread</a>.<img alt="" height="1" src="https://github.com/notifications/beacon/ABWnLb8Ye1NUZWfRrTAleL3D8IArvI5Iks5qqklcgaJpZM4J-dnj.gif" width="1" /></p>
<div itemscope itemtype="http://schema.org/EmailMessage">
<div itemprop="action" itemscope itemtype="http://schema.org/ViewAction">
  <link itemprop="url" href="https://github.com/openstreetmap/openstreetmap-website/pull/1290#issuecomment-247539093"></link>
  <meta itemprop="name" content="View Pull Request"></meta>
</div>
<meta itemprop="description" content="View this Pull Request on GitHub"></meta>
</div>

<script type="application/json" data-scope="inboxmarkup">{"api_version":"1.0","publisher":{"api_key":"05dde50f1d1a384dd78767c55493e4bb","name":"GitHub"},"entity":{"external_key":"github/openstreetmap/openstreetmap-website","title":"openstreetmap/openstreetmap-website","subtitle":"GitHub repository","main_image_url":"https://cloud.githubusercontent.com/assets/143418/17495839/a5054eac-5d88-11e6-95fc-7290892c7bb5.png","avatar_image_url":"https://cloud.githubusercontent.com/assets/143418/15842166/7c72db34-2c0b-11e6-9aed-b52498112777.png","action":{"name":"Open in GitHub","url":"https://github.com/openstreetmap/openstreetmap-website"}},"updates":{"snippets":[{"icon":"PERSON","message":"@paulsturgess in #1290: Hey guys lot's of comments. I can't address them all right at the moment but to pick one:\r\n\r\n\u003e There should be a COPY or ADD instruction to get the local directory to the docker instance running the rails port. I'm not sure on the semantics of copy vs add, so I don't know which is preferred.\r\n\r\nThis isn't required as the `docker-compose.yml` specifies `/app` as the `volume` which exposes the code locally on your machine to make it available in the container.\r\n\r\nThis means you can make changes locally and see them reflected when you refresh the browser.\r\n\r\nThis would obviously be different you wanted to use Docker to deploy to a production server where it would need to clone down the repo. Hopefully that makes sense!"}],"action":{"name":"View Pull Request","url":"https://github.com/openstreetmap/openstreetmap-website/pull/1290#issuecomment-247539093"}}}</script>