trxcllnt commented on a change in pull request #9961:
URL: https://github.com/apache/arrow/pull/9961#discussion_r610381787
##########
File path: js/README.md
##########
@@ -224,7 +224,7 @@ The base `apache-arrow` package includes all the
compilation targets for conveni
The targets are also published under the `@apache-arrow` namespace:
```sh
-npm install apache-arrow # <-- combined es5/UMD,
esnext/CommonJS/ESModules/UMD, and TypeScript package
+npm install apache-arrow # <-- combined es2015/UMD,
esnext/CommonJS/ESModules/UMD, and TypeScript package
Review comment:
While we're editing this line, might as well make it reflect reality too:
```suggestion
npm install apache-arrow # <-- combined es2015/UMD +
esNext/CommonJS/ESModules/UMD
```
##########
File path: js/README.md
##########
@@ -224,7 +224,7 @@ The base `apache-arrow` package includes all the
compilation targets for conveni
The targets are also published under the `@apache-arrow` namespace:
```sh
-npm install apache-arrow # <-- combined es5/UMD,
esnext/CommonJS/ESModules/UMD, and TypeScript package
+npm install apache-arrow # <-- combined es2015/UMD,
esnext/CommonJS/ESModules/UMD, and TypeScript package
Review comment:
While we're editing this line, might as well make it reflect reality too:
```suggestion
npm install apache-arrow # <-- combined es2015/UMD +
esnext/CommonJS/ESModules/UMD
```
--
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:
[email protected]
