Would be nice if the MSI property were more self explanatory like "LOG_TO_EVENTLOG"

When I have to consult the documentation to figure out the purpose of a variable like this it feels like it does not follow "code as documentation" principles well enough. But all the more so when it is a variable that users of will have absolutely no other context for (won't be looking at the code that uses it when they need to use it) - which is the case for all properties fed into MSIs because it is not easy for non-MSI packager to dissect the package to figure out the purpose and in this case it is just a condition on a custom action.