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

Render Message/Enums/Extensions/Services ranges only if present, to a… #410

Merged
merged 4 commits into from
Mar 3, 2020
Merged

Conversation

horacimacias
Copy link

…void extra whitespace which ends up being seen as quote/code block

#365

@pseudomuto
Copy link
Owner

Seems legit. Can you run make examples to update all the things?

@horacimacias
Copy link
Author

I've updated makefile as at least for me it looked like some of the google proto files have been moved from https://github.com/googleapis/googleapis into https://github.com/protocolbuffers/protobuf.
Perhaps I'm missing something here or they are still in https://github.com/googleapis/googleapis in some tag/commit/branch.

After that, I re-generated resources and re-ran examples. The only difference in output for examples is the markdown, as expected, and now the several empty linebreaks are not there.
It's still not 100% free from linebreaks but at least it does not have the 3 it had before.

@pseudomuto
Copy link
Owner

LGTM 👍 Thanks for the PR!

@pseudomuto pseudomuto merged commit 13aae4a into pseudomuto:master Mar 3, 2020
@horacimacias
Copy link
Author

horacimacias commented Mar 3, 2020

perfect! thank you!

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.

None yet

3 participants