rnewson commented on PR #5839: URL: https://github.com/apache/couchdb/pull/5839#issuecomment-4097640606
The comment at https://github.com/apache/couchdb/pull/5792#issuecomment-3715309080; > I agree about @ricellis remarks about overloading types in validate_doc_update and reason. This is easily avoided using different names for these fields for declarative VDUs, e.g. reason remains a string, but detailed mismatch info is in its own field. seems to have been overlooked here? There's a concern that helper libraries in typesafe languages will struggle to cleanly handle this if the value of a validate_doc_update can be a string (for javascript) or an object (for mango). -- 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]
