protoc-gen-doc icon indicating copy to clipboard operation
protoc-gen-doc copied to clipboard

Fix Scalar Value Types Table for Markdown

Open Akronymus-git opened this issue 4 years ago • 8 comments

As per the markdown guidelines you need at least three dashes to have a properly formatted table. As the Go and C# example didn't have it, it caused the table to not render in Rider and Visual Studio. I suspect it will not render in others either.

Fixes #453

Akronymus-git avatar Aug 30 '21 07:08 Akronymus-git

@pseudomuto As per the Contributing.md, I am adding the the ping.

Akronymus-git avatar Aug 31 '21 06:08 Akronymus-git

Hey @ChristophGra, thanks for the PR! Can you please run make examples and add an entry to CHANGELOG.md under the unreleased heading?

pseudomuto avatar Sep 08 '21 17:09 pseudomuto

I must admit that I haven't worked with go before, so I'd appreciate it if you could point me to where I find the documentation on how to do that. I tried using go run but it gave me the error that it is not a main package. Furthermore, when I tried to run it via protoc, even though I ran go install on the package, I didn't manage to get it to use the new version.

@pseudomuto thanks for your time in advance.

Akronymus-git avatar Sep 08 '21 18:09 Akronymus-git

Ok, @pseudomuto I got it to locally generate from the new template by passing it as a doc_opt and it has generated correct markdown, which rendered properly in VSCode, as well as Rider and Visual Studio. Additionally I pushed a change to add the entry to the changelog.

Akronymus-git avatar Sep 08 '21 19:09 Akronymus-git

@ChristophGra can you rebase this?

tmc avatar Jan 19 '22 23:01 tmc

@tmc like that? I hope I did the rebase right.

Akronymus-git avatar Jan 22 '22 22:01 Akronymus-git

@ChristophGra this looks 👌 Can you run make examples or make docker_test to ensure that the generated files are updated (I get a diff when I run it on this branch).

Once that's done we can get this merged....FINALLY 😬

pseudomuto avatar Feb 15 '22 02:02 pseudomuto

The Makefile has changed a bit. Running either make build/examples or make test/docker after a rebase should get the example files sorted out.

pseudomuto avatar Feb 18 '22 17:02 pseudomuto