jkevan commented on code in PR #4: URL: https://github.com/apache/unomi-tracker/pull/4#discussion_r966859198
########## dist/apache-unomi-tracker.cjs.js: ########## @@ -0,0 +1,1571 @@ +'use strict'; Review Comment: Authoring libraries is always a pain if you try to maximize the number of projects that can incorporate yours. In a perfect world, we would have to use only one module system, most probably ES modules as that is the standard in Javascript specifications, but as we might know we don't live in a perfect world and transitioning projects takes a lot of time. So, even though now, NodeJS already has pretty good support for ES modules and all modern browsers are compatible with it a lot of projects still use CJS or UMD or even AMD modules. -- 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]
