flimzy commented on code in PR #712: URL: https://github.com/apache/couchdb-documentation/pull/712#discussion_r851161054
########## src/cluster/nodes.rst: ########## @@ -37,7 +37,7 @@ the nodes it is connected to and knows about. "no...@xxx.xxx.xxx.xxx"] } -* ``all_nodes`` are all the nodes thats this node knows about. +* ``all_nodes`` are all the nodes that's this node knows about. Review Comment: This was the wrong correction :) ```suggestion * ``all_nodes`` are all the nodes that this node knows about. ``` ########## ext/linter.py: ########## @@ -154,13 +154,13 @@ def whitespace_committee(file): in files. The documentation style guide says: - There should be no trailing white space; - - More than one emtpy lines are not allowed and there shouldn't be such + - More than one empty lines are not allowed and there shouldn't be such Review Comment: It might be nice to fix the subject/verb agreement while we're here, too. ```suggestion - More than one empty line is not allowed and there shouldn't be such ``` ########## ext/linter.py: ########## @@ -154,13 +154,13 @@ def whitespace_committee(file): in files. The documentation style guide says: - There should be no trailing white space; - - More than one emtpy lines are not allowed and there shouldn't be such + - More than one empty lines are not allowed and there shouldn't be such at the end of file; - The last line should ends with newline character Additionally it alerts about for tabs if they were used instead of spaces. Review Comment: ```suggestion Additionally it alerts about tabs if they were used instead of spaces. ``` -- 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. To unsubscribe, e-mail: notifications-unsubscr...@couchdb.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org