zentol commented on a change in pull request #13391:
URL: https://github.com/apache/flink/pull/13391#discussion_r495815969



##########
File path: docs/README.md
##########
@@ -126,14 +126,14 @@ You can exclude a heading from the table of contents:
 
        {% top %}
 
-This will be replaced by a default back to top link. It is recommended to use 
these links at least at the end of each level-2 section.
+This will be replaced by default back to top link. It is recommended to use 
these links at least at the end of each level-2 section.

Review comment:
       I'm not really convinced. It still sounds wrong, and it strikes be as 
odd to remove it here despite being present for labels. I will revert this 
change (actually adjust it slightly to remove `default` because I have no idea 
what that is supposed to mean).

##########
File path: docs/README.md
##########
@@ -126,14 +126,14 @@ You can exclude a heading from the table of contents:
 
        {% top %}
 
-This will be replaced by a default back to top link. It is recommended to use 
these links at least at the end of each level-2 section.
+This will be replaced by default back to top link. It is recommended to use 
these links at least at the end of each level-2 section.

Review comment:
       ```suggestion
   This will be replaced by a back to top link. It is recommended to use these 
links at least at the end of each level-2 section.
   ```




----------------------------------------------------------------
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:
[email protected]


Reply via email to