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.

Sign up for GitHub

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

Jump to bottom

Need a configuration element to override MAINTAINER generated in the Dockerfile #117

Closed
xcurry opened this issue Mar 5, 2015 · 2 comments
Closed

Need a configuration element to override MAINTAINER generated in the Dockerfile #117

xcurry opened this issue Mar 5, 2015 · 2 comments
Assignees
@rhuss
Labels
fixed
Milestone
0.11.3

Comments

@xcurry
Copy link

xcurry commented Mar 5, 2015

Looks like this field is currently hardcoded.
@rhuss rhuss added this to the 0.11.3 milestone Mar 6, 2015
@rhuss rhuss self-assigned this Mar 6, 2015
@akobiakov akobiakov mentioned this issue Mar 18, 2015
Closed
@akobiakov
Copy link

Hi, Roland. Thank you for your cool plugin!
I took this task as a small exercise in contributing, please find my PR above.

@rhuss
Copy link
Collaborator

rhuss commented Mar 18, 2015

Thanks a lot, I will have a look at the PR tomorrow.

rhuss added a commit that referenced this issue Mar 20, 2015
@rhuss
Add "maintainer" as keys for property based configuration.
f33c05f 
Related to #117 and #129
@rhuss rhuss added the fixed label Mar 28, 2015
@rhuss rhuss closed this as completed in 1fd9273 Apr 21, 2015
leusonmario pushed a commit to leusonmario/docker-maven-plugin that referenced this issue Aug 18, 2018
@davidxia
Merge pull request fabric8io#117 from spotify/checkstyle
57368c2 
Fix code style
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@rhuss rhuss

Labels
fixed
Projects
None yet
Milestone
0.11.3
Development

Successfully merging a pull request may close this issue.

Allow to define a maintainer explicitly
3 participants
@rhuss @xcurry @akobiakov