onitake commented on issue #296: URL: https://github.com/apache/cloudstack-primate/pull/296#issuecomment-617825950
I tested b3805f8 and it solved the issue reported in #294 . I haven't tested 944c5f6 yet. Is there a specific reason why you changed `(x && y) || z` to `x ? y : z`? A general dislike of the ternary operator notwithstanding, I think the ?: construct is easier to read - but YMMV. ---------------------------------------------------------------- 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: us...@infra.apache.org
