jaydoane commented on PR #4287: URL: https://github.com/apache/couchdb/pull/4287#issuecomment-1336273945
Just a couple nits: First, the title of the PR "Allow = in config key values" makes it sound like we're allowing the `=` character in the _values_, but I think you mean "key names"? > There are allowed only for the `k = v` form and not `k=v`. Do you mean "They are allowed ..."? > The idea is to split on `=` first, and if that fails to produce a valid kv pair we split on `=` as before. Should the first one contain spaces on either side of the equal sign, like ` = `? > Also added 100% test coverage to the parser. 💯 awesome -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
