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

Add empheral dockerfile handling #2456

Closed
wants to merge 1 commit into from
Closed

Add empheral dockerfile handling #2456

wants to merge 1 commit into from

Conversation

majkrzak
Copy link

@majkrzak majkrzak commented Nov 5, 2019

close #2105 according to #2105 (comment)

@GordonTheTurtle
Copy link

Please sign your commits following these rules:
https://github.com/moby/moby/blob/master/CONTRIBUTING.md#sign-your-work
The easiest way to do this is to amend the last commit:

$ git clone -b "patch-1" git@github.com:majkrzak/docker-py.git somewhere
$ cd somewhere
$ git commit --amend -s --no-edit
$ git push -f

Amending updates the existing PR. You DO NOT need to open a new one.

@majkrzak
Copy link
Author

majkrzak commented Nov 5, 2019

🤦‍♂️

close #2105 according to #2105 (comment)

Signed-off-by: Piotr Majkrzak <piotr@majkrzak.dev>
@majkrzak
Copy link
Author

majkrzak commented Feb 9, 2020

👋 are you fine down there? 👋

@majkrzak
Copy link
Author

👋

@majkrzak
Copy link
Author

I wonder if this PR will get merged faster than gkoberger/stacksort#4

@ulyssessouza
Copy link
Collaborator

Hello @majkrzak. I think that trying to detect if it's the content or not is not an accurate way.
For example, this is a valid Dockerfile content AND a filename

FROM ubuntu

IMO another parameter with the content would be better

@majkrzak majkrzak closed this by deleting the head repository Nov 1, 2022
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.

Can not pass fileobj together with path context to build
3 participants