Skip to content

BAEL-4316 Sample apps to showcase how to add files from outside docker context#11606

Merged
ashleyfrieze merged 7 commits into
eugenp:masterfrom
swapanpramanick2004:BAEL-4316
Feb 22, 2022
Merged

BAEL-4316 Sample apps to showcase how to add files from outside docker context#11606
ashleyfrieze merged 7 commits into
eugenp:masterfrom
swapanpramanick2004:BAEL-4316

Conversation

@swapanpramanick2004

@swapanpramanick2004 swapanpramanick2004 commented Dec 20, 2021

Copy link
Copy Markdown
Contributor

BAEL-4316 Sample apps to showcase how to add files from outside docker context

  • Sample App
  • Dockerfiles
  • Docker-compose file
  • Scripts

Comment thread docker/docker-sample-app/README.md Outdated
### Relevant Articles:

- How to Get Docker-Compose to Always Use the Latest Image
- How to Include Files Outside of Docker’s Build Context

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Let's remove this - we don't update the article list as part of these PRs

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Quick reminder that this needs reverting

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Ohh, sorry, it was actually a mistake. I thought I did this change in the directory docker/docker-sample-ecomm-app and the change would go away when I deleted the directory. But it seems I changed thee wrong file when I made this change in the first place.

Comment thread docker/docker-sample-app/README.md Outdated
### Relevant Articles:

- How to Get Docker-Compose to Always Use the Latest Image
- How to Include Files Outside of Docker’s Build Context

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Quick reminder that this needs reverting

@ashleyfrieze ashleyfrieze merged commit 576b617 into eugenp:master Feb 22, 2022
pingm26 pushed a commit to pingm26/tutorials that referenced this pull request May 6, 2026
…r context (eugenp#11606)

* BAEL-4316 Sample apps to showcase how to add files from outside docker context

* BAEL-4316 update README content

* BAEL-4316 change example

* BAEL-4316 update docker file

* BAEL-4316 revert the README changes

* BAEL-4316 change directory structure

* BAEL-4316 deleted old files

Co-authored-by: Swapan Pramanick <swapan.pramanick@zalando.de>
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.

2 participants