rubenada commented on a change in pull request #2050:
URL: https://github.com/apache/calcite/pull/2050#discussion_r448327076



##########
File path: site/_docs/howto.md
##########
@@ -474,7 +474,7 @@ $ ./src/main/scripts/generate-protobuf.sh
 The following sections are of interest to Calcite committers and in
 particular release managers.
 
-## Managing Calcite repositories through GitHub (for Calcite committers)
+## Managing Calcite repositories through GitHub
 
 Committers have write access to Calcite's 
 [ASF git repositories](https://gitbox.apache.org/repos/asf#calcite) hosting

Review comment:
       I think @zabetak has a point. If we have a section called "Advanced 
topics for committers", then it seems redundant to have every subsection in 
there called "XYZ (for Calcite committers)".
   Right before that we have an "Advanced topics for developers" section and we 
do not have the suffix "(for developers)" on each subsection.




----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


Reply via email to