Skip to content

Conversation

josequaresma
Copy link

The following command wasn't working curl $URL/hello/quarkus since it requires /hello/greeting/quarkus. I have updated that and have done two other small changes to make the output a bit more friendly

@josequaresma josequaresma marked this pull request as ready for review March 30, 2019 19:47
@gsmet
Copy link
Member

gsmet commented Apr 2, 2019

@josequaresma sorry for not replying earlier. In fact the reference files for the guides are in the main repo here: https://github.com/quarkusio/quarkus/tree/master/docs/src/main/asciidoc and are synced when we release (I added a warning at the top of the files for the next release).

Could you submit your PR there? I'll apply it right away? We release a new version tomorrow so if I can merge it by then, it will be in the next release.

@gsmet gsmet closed this Apr 2, 2019
@josequaresma
Copy link
Author

Ah! Sorry for mixing it up @gsmet . Will submit the PR there in a couple of hours

@gsmet
Copy link
Member

gsmet commented Apr 2, 2019

Yeah, it was not very clear until now, hopefully the message at the top will help (it will be deployed tomorrow). Thanks!

@josequaresma
Copy link
Author

@gsmet PR (hopefully the right one now) created here quarkusio/quarkus#1823

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