You are viewing a plain text version of this content. The canonical link for it is here.

Posted to notifications@couchdb.apache.org by GitBox <gi...@apache.org> on 2021/06/23 20:44:16 UTC

[GitHub] [couchdb] chewbranca commented on issue #3473: Normalise all config options so default.ini is 100% commented out

chewbranca commented on issue #3473:
URL: https://github.com/apache/couchdb/issues/3473#issuecomment-867145131


   If the aim is to get `default.ini` to be fully commented, then I think it would be prudent to delete `config:get/2` [1] which injects a default value in. If we're expecting to have the config options fully defined in code, then i don't think there is a scenario where having a config value lookup but not having a default valued defined in the code is appropriate.
   
   [1] https://github.com/apache/couchdb-config/blob/main/src/config.erl#L156-L157


-- 
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:
users@infra.apache.org