[GitHub] [arrow] domoritz commented on pull request #9961: ARROW-12309: [JS] Make es2015 bundles the default

2021-04-11 Thread GitBox
domoritz commented on pull request #9961: URL: https://github.com/apache/arrow/pull/9961#issuecomment-817366232 I didn't mean not having bundles. I mean that the bundle will default to es2015. The es5 bundle wouldn't be as optimal anymore but the es5 bundles don't seem to get much traffic

[GitHub] [arrow] domoritz commented on pull request #9961: ARROW-12309: [JS] Make es2015 bundles the default

2021-04-11 Thread GitBox
domoritz commented on pull request #9961: URL: https://github.com/apache/arrow/pull/9961#issuecomment-817342595 I wonder, would it make sense to remove closure compiler if we only use it for es5 and es5 isn't the default bundle anymore? Is the benefit for users big enough to warrant the

[GitHub] [arrow] domoritz commented on pull request #9961: ARROW-12309: [JS] Make es2015 bundles the default

2021-04-08 Thread GitBox
domoritz commented on pull request #9961: URL: https://github.com/apache/arrow/pull/9961#issuecomment-816415923 I think I need your help, @trxcllnt, with the closure compiler here. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to