digitalextremist commented on PR #358: URL: https://github.com/apache/couchdb-nano/pull/358#issuecomment-3291650551
Removed `WIP` and here is the follow-up reasoning from someone who looks both ways when crossing one-way streets: In the hypothetical, there is the possibility that `response.message` does not exist; versus in the change as proposed, we know for sure `response.message` is there and not `null` I have tried to move away from `||` over to `??` if possible, in general because `||` is 'falsey` versus `??` which is 'nullish' which is what we are looking for in this case. Same with `?` there... Looking ahead 10 years: This is _almost_ to the point of being able to use `??=` but not quite; likely after a refactor... until then, the newer operations are also smaller in token-size and download size after minification. -- 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]
