Skip to content

Conversation

@michaellandi
Copy link
Contributor

@michaellandi michaellandi commented Nov 27, 2024

[POST RE:INVENT]

Problem

Nested Dockerfile files were not supported with #6107

Solution

Look at path basename when comparing file to list of well known files.

I manually tested adding nested Dockerfiles to various folders and ensured that pattern matched.


License: I confirm that my contribution is made under the terms of the Apache 2.0 license.

@github-actions
Copy link

  • This pull request modifies code in src/* but no tests were added/updated.
    • Confirm whether tests should be added or ensure the PR description explains why tests are not required.

@michaellandi michaellandi marked this pull request as ready for review November 27, 2024 20:12
@michaellandi michaellandi requested a review from a team as a code owner November 27, 2024 20:12
@michaellandi michaellandi changed the title fix(amazonq): /dev to support upload of nested dockerfiles [POST RE:INVENT] fix(amazonq): /dev to support upload of nested dockerfiles Nov 27, 2024
@michaellandi michaellandi changed the title [POST RE:INVENT] fix(amazonq): /dev to support upload of nested dockerfiles fix(amazonq): /dev to support upload of nested dockerfiles Nov 27, 2024
@justinmk3 justinmk3 disabled auto-merge December 6, 2024 20:53
@justinmk3 justinmk3 merged commit 33b791e into aws:master Dec 6, 2024
23 of 25 checks passed
karanA-aws pushed a commit to karanA-aws/aws-toolkit-vscode that referenced this pull request Jan 17, 2025
Nested `Dockerfile` files were not supported with
aws#6107

Look at path basename when comparing file to list of well known files.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants