[GitHub] [couchdb-documentation] flimzy commented on a diff in pull request #732: Add missing description and fix typos

[GitBox <gi...@apache.org>]

flimzy commented on code in PR #732:
URL: https://github.com/apache/couchdb-documentation/pull/732#discussion_r874469268


##########
src/config/http.rst:
##########
@@ -129,7 +129,7 @@ HTTP Server Options
             [chttpd]
             changes_timeout = 60000 ; 60 seconds
 
-    .. config:option:: config_whitelist :: Config options while list
+    .. config:option:: config_whitelist :: Config options white list

Review Comment:
   I think it should also be a single word, to be consistent with the following paragraph.
   
   ```suggestion
       .. config:option:: config_whitelist :: Config options whitelist
   ```



##########
src/config/http.rst:
##########
@@ -129,7 +129,7 @@ HTTP Server Options
             [chttpd]
             changes_timeout = 60000 ; 60 seconds
 
-    .. config:option:: config_whitelist :: Config options while list
+    .. config:option:: config_whitelist :: Config options white list

Review Comment:
   I'm also a bit surprised to see the word `whitelist` still used within CouchDB. We made an effort to remove certain words in the past, but it looks like this one didn't get caught.



-- 
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-unsubscribe@couchdb.apache.org

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