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



##########
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:
       > The whole section is about committers (see line 472) and the suffix is 
repeated in every subsection so IMHO it is redundant. It makes headers and 
links longer without adding new information.
   
   Makes sense. Thanks for your explanation.




----------------------------------------------------------------
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