Support for build-args of docker build#410
Conversation
In 1.9 there is new property - build-args that can be specified for docker build command
There was a problem hiding this comment.
There was a problem hiding this comment.
That's docker version, not docker API https://docs.docker.com/engine/reference/api/docker_remote_api/
There was a problem hiding this comment.
Could you add javadoc? Example:
and reference to field
|
I have added javadoc with api version information |
|
LGTM, @marcuslinke ? |
|
@alenkacz @KostyaSha I'm OK with it but it needs to be implemented/tested for the netty based |
|
I have added that netty based implementation. By the way - keeping two implementations like this is not a good architectural decision - I think. It is really error prone... Also I think that you should detach this forked repository if you are not planning on merging it back to the previous one. Because if it is like this, contributions are not counted for example - because they are counted only after fork is merged to the original repository. Which will never happen. How to detach fork is written here - https://www.quora.com/Git-revision-control/How-can-one-detach-a-forked-project-in-GitHub I used GitHub support in the past and they are really quick. |
|
@alenkacz Thanks! The netty implementation will supersede the old jersey implementation in the near future so this is a temporary situation only that will stay until after the next release ( > v3.0.0). I will try to reach Github support to detach the repository as you suggested. Didn't know this is possible. Will merge this ASAP. Thanks for contributing! |
Fork shows history, initial repository is not used anymore and technically i think it in mergeable state. |
|
@marcuslinke does this fork changed artifactId? If not then i see no reasons for breaking fork chain. |
|
@KostyaSha The groupId was changed from |
|
Let's try. |
|
Done. |
3 participants
In 1.9 there is new property - build-args that can be specified for
docker build command