Skip to content

Conversation

geoand
Copy link
Contributor

@geoand geoand commented Mar 15, 2019

Fixes: #1498

Copy link
Member

@gsmet gsmet left a comment

Choose a reason for hiding this comment

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

Looks good to me apart from the hardcoded properties.

@geoand geoand force-pushed the wrapper-in-generated-projects branch from f581b36 to 97ea114 Compare March 15, 2019 10:37
@geoand
Copy link
Contributor Author

geoand commented Mar 15, 2019

@gsmet Hardcoded values removed

@geoand geoand requested a review from gsmet March 15, 2019 10:37
@geoand geoand force-pushed the wrapper-in-generated-projects branch from 97ea114 to 84b7d58 Compare March 15, 2019 11:21
@geoand geoand requested a review from gsmet March 15, 2019 11:21
@geoand
Copy link
Contributor Author

geoand commented Mar 15, 2019

@gsmet OK to merge now?

@geoand geoand force-pushed the wrapper-in-generated-projects branch from 84b7d58 to 230321b Compare March 15, 2019 22:33
@geoand
Copy link
Contributor Author

geoand commented Mar 15, 2019

PR rebased onto latest master to fix a conflict in the rest client guide

@geoand geoand force-pushed the wrapper-in-generated-projects branch from 230321b to 74f6f69 Compare March 15, 2019 22:37
@gsmet gsmet merged commit 03bd386 into quarkusio:master Mar 16, 2019
@gsmet gsmet added this to the 0.12.0 milestone Mar 16, 2019
@geoand geoand deleted the wrapper-in-generated-projects branch March 17, 2019 16:18
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.

2 participants