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

verbose parameter was converted to a String rather than a boolean value. #14

Merged
merged 3 commits into from
Jun 19, 2019

Conversation

rohanKanojia
Copy link
Contributor

@rohanKanojia rohanKanojia commented Jun 18, 2019

Changes in Plugins according to changes in kit. See
fabric8io/docker-maven-plugin#917

These changes are required after merging jshiftio/jshift-kit#42

@@ -44,7 +44,7 @@

// For verbose output
@Parameter(property = "fabric8.verbose", defaultValue = "false")

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

One fabric8 property here, I know it was already there not introduced by this PR but can you fix it ?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yes

@lordofthejars lordofthejars self-requested a review June 18, 2019 16:12
@lordofthejars lordofthejars merged commit 3428841 into jshiftio:master Jun 19, 2019
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.

3 participants