-
-
Notifications
You must be signed in to change notification settings - Fork 2k
filter incompatible docker option when building dockerfile vs herokuish apps #1712
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
Conversation
|
Is What about dockerfile plugins that don't work for herokuish builds like |
|
Probably not and it is short-sighted indeed. However, leaving this broken is probably not good either. EDIT: Also I don't follow what plugins you're referring to when you reference |
|
Yeah I was just bringing up the inverse problem where a plugin is expecting a dockerfile deployment. It does seem beneficial to tell the plugin which deployment type is happening to really solve this issue. |
cfe967b to
d0032f2
Compare
|
Updated to take into account the inverse problem. This is by no means exhaustive but gets us a pattern to filter args for now. |
|
@michaelshobbs you want to add that static buildpack and a test app to herokuish? |
|
I can yeah. My fork or the official one? |
|
Your fork. Or just move your fork to the dokku org and be done with it. Official fork? |
|
Yeah I'll revert this change here and open a new PR |
f512120 to
d0032f2
Compare
d0032f2 to
2c34a60
Compare
filter incompatible docker option when building dockerfile vs herokuish apps
i'm not sure this is the best solution but it works in leiu of making the plugins handle it
closes #1609