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

fix(pipeline): buildspec works with dockerfiles in subdirs #902

Merged
merged 2 commits into from
May 6, 2020
Merged

fix(pipeline): buildspec works with dockerfiles in subdirs #902

merged 2 commits into from
May 6, 2020

Conversation

efekarakus
Copy link
Contributor

Fixes #893

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@efekarakus efekarakus added the type/bug Issues that are bugs. label May 6, 2020
@efekarakus
Copy link
Contributor Author

Screenshot showing that two applications fe which sits at the root, and rindex that's in a subdirectory deploys:
Screen Shot 2020-05-06 at 12 20 15 PM

in the future, we can add an e2e test that follows this file hierarchy

@mergify mergify bot merged commit f87b156 into aws:master May 6, 2020
@msafder
Copy link

msafder commented May 6, 2020

Pulled in this change manually and tried it out and it work like a charm, thanks for great solution 👍🏻

@efekarakus efekarakus deleted the issues/893 branch May 6, 2020 23:32
bvtujo pushed a commit to bvtujo/copilot-cli that referenced this pull request May 12, 2020
Fixes aws#893

Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type/bug Issues that are bugs.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

CodePipeline build step fails if app's Dockerfile is not in root
4 participants