Closed aaliddell closed 2 years ago
high level questions - yeah I agree the GH template needs some work, it's too narrow - let's leave that presentation bit until after we have the markdown content.
IMO there's no one to dictate a style guide and I appreciate the voice of the author so whatever dialect you prefer. I suppose a good tech writer would modify some words here and there for wider readability. Maybe the SIG can attract one of them someday.
IMO there's no one to dictate a style guide and I appreciate the voice of the author so whatever dialect you prefer. I suppose a good tech writer would modify some words here and there for wider readability. Maybe the SIG can attract one of them someday.
I dont mind either way, so I've switched the obvious cases to US English, since that's generally the biggest userbase.
I reviewed the document. I could imagine more content in the areas of:
FileDescriptorSet
with proto_library
and descriptor_set_in
.However, those things could be added later, I don't think adding new documentation scope is necessary to merge this.
Thanks @aaliddell for working on this!
For the theme, how about remote_theme: pmarsceill/just-the-docs
https://alexeagle.github.io/SIG-rules-authors/proto-grpc.html
seems popular from some searching, and it's wider than the current one
For the theme, how about
remote_theme: pmarsceill/just-the-docs
https://alexeagle.github.io/SIG-rules-authors/proto-grpc.html
seems popular from some searching, and it's wider than the current one
Look OK to me
To tie up all the things left open:
--experimental_proto_descriptor_sets_include_source_info
when using documentation plugins.Yup I can fix the index page, or someone else can take the content from the current readme and just move it to GH-pages branch
Hey @aaliddell do you think we can merge this as-is, and you can invoice your hours to the SIG open collective?
Fine by me
A WiP document guiding users on the use of Protobuf and gRPC with Bazel, following from https://github.com/bazel-contrib/SIG-rules-authors/discussions/26 and other discussions on mailing list.
You can see a live version of the page here for review: https://aaliddell.github.io/SIG-rules-authors/proto-grpc
This is still a draft, as there are a few outstanding points:
Table is not embedding correctlyIs American English preferred? e.g colour vs color, summarises vs summarizes etc