chore(commonjs): adapt to updated dependencies

rollup · Jul 31, 2020 · c2fb584 · c2fb584
1 parent 3364284
commit c2fb584
Show file tree

Hide file tree

Showing 46 changed files with 84 additions and 73 deletions.
diff --git a/codecov.yml b/codecov.yml
@@ -5,6 +5,6 @@ coverage:
   round: down
   range: 70...100
   status:
-    project: 'no'
-    patch: 'yes'
-comment: 'off'
+    project: "no"
+    patch: "yes"
+comment: "off"
diff --git a/package.json b/package.json
@@ -7,7 +7,7 @@
     "ci:lint": "node scripts/run-changed.js ci:lint",
     "ci:test": "node scripts/run-changed.js ci:test",
     "lint": "pnpm run lint:docs && npm run lint:json && pnpm run lint:package && node scripts/run-changed.js lint",
-    "lint:docs": "prettier --single-quote --arrow-parens avoid --write README.md",
+    "lint:docs": "prettier --single-quote --arrow-parens avoid --trailing-comma none --write README.md",
     "lint:js": "eslint scripts util",
     "lint:json": "prettier --write tsconfig.*.json codecov.yml pnpm-workspace.yaml",
     "lint:package": "prettier --write package.json --plugin=prettier-plugin-package",

diff --git a/packages/alias/package.json b/packages/alias/package.json
@@ -22,7 +22,7 @@
     "ci:lint:commits": "commitlint --from=${CIRCLE_BRANCH} --to=${CIRCLE_SHA1}",
     "ci:test": "pnpm run test -- --verbose",
     "lint": "pnpm run lint:js && pnpm run lint:docs && pnpm run lint:package",
-    "lint:docs": "prettier --single-quote --arrow-parens avoid --write README.md",
+    "lint:docs": "prettier --single-quote --arrow-parens avoid --trailing-comma none --write README.md",
     "lint:js": "eslint --fix --cache src test types --ext .js,.ts",
     "lint:package": "prettier --write package.json --plugin=prettier-plugin-package",
     "prebuild": "del-cli dist",

diff --git a/packages/auto-install/package.json b/packages/auto-install/package.json
@@ -19,7 +19,7 @@
     "ci:lint:commits": "commitlint --from=${CIRCLE_BRANCH} --to=${CIRCLE_SHA1}",
     "ci:test": "pnpm run test -- --verbose",
     "lint": "pnpm run lint:js && pnpm run lint:docs && pnpm run lint:package",
-    "lint:docs": "prettier --single-quote --arrow-parens avoid --write README.md",
+    "lint:docs": "prettier --single-quote --arrow-parens avoid --trailing-comma none --write README.md",
     "lint:js": "eslint --fix --cache src test --ext .js,.ts",
     "lint:package": "prettier --write package.json --plugin=prettier-plugin-package",
     "prebuild": "del-cli dist",
@@ -54,9 +54,7 @@
   },
   "types": "types/index.d.ts",
   "ava": {
-    "babel": {
-      "compileEnhancements": false
-    },
+    "babel": true,
     "extensions": [
       "ts"
     ],

diff --git a/packages/auto-install/test/npm-bare.js b/packages/auto-install/test/npm-bare.js
@@ -15,7 +15,7 @@ const input = join(cwd, '../input.js');
 process.chdir(cwd);
 
 test('npm, bare', async (t) => {
-  t.timeout(30000);
+  t.timeout(50000);
   await rollup({
     input,
     output: {

diff --git a/packages/auto-install/test/npm.js b/packages/auto-install/test/npm.js
@@ -17,7 +17,7 @@ const pkgFile = join(cwd, 'package.json');
 process.chdir(cwd);
 
 test('invalid manager', (t) => {
-  t.timeout(30000);
+  t.timeout(50000);
   const error = t.throws(
     () =>
       rollup({
@@ -36,7 +36,7 @@ test('invalid manager', (t) => {
 });
 
 test('npm', async (t) => {
-  t.timeout(30000);
+  t.timeout(50000);
   await rollup({
     input,
     output: {

diff --git a/packages/auto-install/test/yarn-bare.js b/packages/auto-install/test/yarn-bare.js
@@ -15,7 +15,7 @@ const input = join(cwd, '../input.js');
 process.chdir(cwd);
 
 test('yarn, bare', async (t) => {
-  t.timeout(30000);
+  t.timeout(50000);
   await rollup({
     input,
     output: {

diff --git a/packages/auto-install/test/yarn.js b/packages/auto-install/test/yarn.js
@@ -15,7 +15,7 @@ const input = join(cwd, '../input.js');
 process.chdir(cwd);
 
 test('yarn', async (t) => {
-  t.timeout(30000);
+  t.timeout(50000);
   await rollup({
     input,
     output: {

diff --git a/packages/babel/package.json b/packages/babel/package.json
@@ -22,7 +22,7 @@
     "ci:lint:commits": "commitlint --from=${CIRCLE_BRANCH} --to=${CIRCLE_SHA1}",
     "ci:test": "pnpm run test -- --verbose",
     "lint": "pnpm run lint:js && pnpm run lint:docs && pnpm run lint:package",
-    "lint:docs": "prettier --single-quote --arrow-parens avoid --write README.md",
+    "lint:docs": "prettier --single-quote --arrow-parens avoid --trailing-comma none --write README.md",
     "lint:js": "eslint --fix --cache src test",
     "lint:package": "prettier --write package.json --plugin=prettier-plugin-package",
     "prebuild": "del-cli dist",

diff --git a/packages/beep/package.json b/packages/beep/package.json
@@ -20,7 +20,7 @@
     "ci:lint:commits": "commitlint --from=${CIRCLE_BRANCH} --to=${CIRCLE_SHA1}",
     "ci:test": "ava --verbose",
     "lint": "pnpm run lint:js && pnpm run lint:docs && pnpm run lint:package",
-    "lint:docs": "prettier --single-quote --arrow-parens avoid --write README.md",
+    "lint:docs": "prettier --single-quote --arrow-parens avoid --trailing-comma none --write README.md",
     "lint:js": "eslint --fix --cache lib test types --ext .js,.ts",
     "lint:package": "prettier --write package.json --plugin=prettier-plugin-package",
     "test": "ava"

diff --git a/packages/buble/package.json b/packages/buble/package.json
@@ -22,7 +22,7 @@
     "ci:lint:commits": "commitlint --from=${CIRCLE_BRANCH} --to=${CIRCLE_SHA1}",
     "ci:test": "pnpm run test -- --verbose",
     "lint": "pnpm run lint:js && pnpm run lint:docs && pnpm run lint:package",
-    "lint:docs": "prettier --single-quote --arrow-parens avoid --write README.md",
+    "lint:docs": "prettier --single-quote --arrow-parens avoid --trailing-comma none --write README.md",
     "lint:js": "eslint --fix --cache src test types --ext .js,.ts",
     "lint:package": "prettier --write package.json --plugin=prettier-plugin-package",
     "prebuild": "del-cli dist",

diff --git a/packages/commonjs/README.md b/packages/commonjs/README.md
@@ -34,9 +34,9 @@ export default {
   input: 'src/index.js',
   output: {
     dir: 'output',
-    format: 'cjs'
+    format: 'cjs',
   },
-  plugins: [commonjs()]
+  plugins: [commonjs()],
 };
 ```
 
@@ -66,8 +66,8 @@ commonjs({
     '!node_modules/logform/index.js',
     '!node_modules/logform/format.js',
     '!node_modules/logform/levels.js',
-    '!node_modules/logform/browser.js'
-  ]
+    '!node_modules/logform/browser.js',
+  ],
 });
 ```
 
@@ -157,7 +157,7 @@ import * as foo from 'foo';
 ```
 
 This is in line with how other bundlers handle this situation and is also the most likely behaviour in case Node should ever support this. However there are some situations where this may not be desired:
-  
+
 - There is code in an external dependency that cannot be changed where a `require` statement expects the default export to be returned from an ES module.
 - If the imported module is in the same bundle, Rollup will generate a namespace object for the imported module which can increase bundle size unnecessarily:
 
@@ -168,15 +168,15 @@ This is in line with how other bundlers handle this situation and is also the mo
 
   // input: dep.js
   export default 'foo';
-  
+
   // output
   var dep = 'foo';
-  
-  var dep$1 = /*#__PURE__*/Object.freeze({
-  	__proto__: null,
-  	'default': dep
+
+  var dep$1 = /*#__PURE__*/ Object.freeze({
+    __proto__: null,
+    default: dep,
   });
-  
+
   console.log(dep$1.default);
   ```
 
@@ -191,22 +191,26 @@ For these situations, you can change Rollup's behaviour either globally or per m
 
   // output
   import * as dep from 'dep';
-  
+
   console.log(dep);
   ```
-  
+
 - `"auto"`: This is complementary to how [`output.exports`](https://rollupjs.org/guide/en/#outputexports): `"auto"` works in Rollup: If a module has a default export and no named exports, requiring that module returns the default export. In all other cases, the namespace is returned. For external dependencies when using `esmExternals: true`, a corresponding interop helper is added:
 
   ```js
   // output
   import * as dep$1 from 'dep';
-
-  function getDefaultExportFromNamespaceIfNotNamed (n) {
-    return n && Object.prototype.hasOwnProperty.call(n, 'default') && Object.keys(n).length === 1 ? n['default'] : n;
+
+  function getDefaultExportFromNamespaceIfNotNamed(n) {
+    return n &&
+      Object.prototype.hasOwnProperty.call(n, 'default') &&
+      Object.keys(n).length === 1
+      ? n['default']
+      : n;
   }
-  
+
   var dep = getDefaultExportFromNamespaceIfNotNamed(dep$1);
-  
+
   console.log(dep);
   ```
 
@@ -215,13 +219,15 @@ For these situations, you can change Rollup's behaviour either globally or per m
   ```js
   // output
   import * as dep$1 from 'dep';
-
-  function getDefaultExportFromNamespaceIfPresent (n) {
-    return n && Object.prototype.hasOwnProperty.call(n, 'default') ? n['default'] : n;
+
+  function getDefaultExportFromNamespaceIfPresent(n) {
+    return n && Object.prototype.hasOwnProperty.call(n, 'default')
+      ? n['default']
+      : n;
   }
-  
+
   var dep = getDefaultExportFromNamespaceIfPresent(dep$1);
-  
+
   console.log(dep);
   ```
 
@@ -230,7 +236,7 @@ For these situations, you can change Rollup's behaviour either globally or per m
   ```js
   // output
   import dep from 'dep';
-  
+
   console.log(dep);
   ```
 
@@ -250,9 +256,9 @@ export default {
   output: {
     file: 'bundle.js',
     format: 'iife',
-    name: 'MyModule'
+    name: 'MyModule',
   },
-  plugins: [resolve(), commonjs()]
+  plugins: [resolve(), commonjs()],
 };
 ```
 
@@ -262,7 +268,7 @@ Symlinks are common in monorepos and are also created by the `npm link` command.
 
 ```js
 commonjs({
-  include: /node_modules/
+  include: /node_modules/,
 });
 ```
 

diff --git a/packages/commonjs/package.json b/packages/commonjs/package.json
@@ -22,7 +22,7 @@
     "ci:lint:commits": "commitlint --from=${CIRCLE_BRANCH} --to=${CIRCLE_SHA1}",
     "ci:test": "pnpm run test -- --verbose && pnpm run test:ts",
     "lint": "pnpm run lint:js && pnpm run lint:docs && pnpm run lint:package",
-    "lint:docs": "prettier --single-quote --arrow-parens avoid --write README.md",
+    "lint:docs": "prettier --single-quote --arrow-parens avoid --trailing-comma none --write README.md",
     "lint:js": "eslint --fix --cache src test types --ext .js,.ts",
     "lint:package": "prettier --write package.json --plugin=prettier-plugin-package",
     "prebuild": "del-cli dist",

diff --git a/packages/commonjs/test/fixtures/function/import-esm-require-returns-default-true/main.js b/packages/commonjs/test/fixtures/function/import-esm-require-returns-default-true/main.js
@@ -1,6 +1,7 @@
 const externalNamed = require('external-esm-named');
 const externalMixed = require('external-esm-mixed');
 const externalDefault = require('external-esm-default');
+
 const mixedExports = require('./mixed.js');
 const defaultExport = require('./default.js');
 

diff --git a/packages/commonjs/test/fixtures/function/require-esm-with-namespace-named-default/dep.js b/packages/commonjs/test/fixtures/function/require-esm-with-namespace-named-default/dep.js
@@ -1 +1,3 @@
-export * as default from './other.js';
+import * as other from './other.js';
+
+export default other;
diff --git a/packages/commonjs/test/fixtures/function/transpiled-esm-default/main.js b/packages/commonjs/test/fixtures/function/transpiled-esm-default/main.js
@@ -1,4 +1,5 @@
 import external from 'external-esm-default';
+
 import dep from './dep';
 
 t.is(dep, 'default');

diff --git a/packages/commonjs/test/fixtures/function/transpiled-esm-mixed/main.js b/packages/commonjs/test/fixtures/function/transpiled-esm-mixed/main.js
@@ -1,4 +1,5 @@
 import external, { foo } from 'external-esm-mixed';
+
 import dep, { named } from './dep';
 
 t.is(dep, 'default');

diff --git a/packages/commonjs/test/fixtures/function/transpiled-esm-named/main.js b/packages/commonjs/test/fixtures/function/transpiled-esm-named/main.js
@@ -1,4 +1,5 @@
 import { foo } from 'external-esm-named';
+
 import { named } from './dep';
 
 t.is(named, 'named');

diff --git a/packages/commonjs/test/fixtures/function/transpiled-esm-namespace-default/main.js b/packages/commonjs/test/fixtures/function/transpiled-esm-namespace-default/main.js
@@ -1,4 +1,5 @@
 import * as external from 'external-esm-default';
+
 import * as dep from './dep';
 
 t.deepEqual(dep, {

diff --git a/packages/commonjs/test/fixtures/function/transpiled-esm-namespace-mixed/main.js b/packages/commonjs/test/fixtures/function/transpiled-esm-namespace-mixed/main.js
@@ -1,4 +1,5 @@
 import * as external from 'external-esm-mixed';
+
 import * as dep from './dep';
 
 t.deepEqual(dep, {

diff --git a/packages/commonjs/test/fixtures/function/transpiled-esm-namespace-named/main.js b/packages/commonjs/test/fixtures/function/transpiled-esm-namespace-named/main.js
@@ -1,4 +1,5 @@
 import * as external from 'external-esm-named';
+
 import * as dep from './dep';
 
 t.deepEqual(dep, {

diff --git a/packages/commonjs/test/snapshots/function.js.md b/packages/commonjs/test/snapshots/function.js.md
@@ -2,7 +2,7 @@
 
 The actual snapshot is saved in `function.js.snap`.
 
-Generated by [AVA](https://ava.li).
+Generated by [AVA](https://avajs.dev).
 
 ## assign-properties-to-default-export
 
@@ -3917,13 +3917,13 @@ Generated by [AVA](https://ava.li).
       const foo = 'foo';␊
       var other = 'bar';␊
       ␊
-      var dep = /*#__PURE__*/Object.freeze({␊
+      var other$1 = /*#__PURE__*/Object.freeze({␊
       	__proto__: null,␊
       	foo: foo,␊
       	'default': other␊
       });␊
       ␊
-      t.deepEqual(dep, { default: 'bar', foo: 'foo' });␊
+      t.deepEqual(other$1, { default: 'bar', foo: 'foo' });␊
       ␊
       var main = {␊
       ␊

diff --git a/packages/commonjs/test/snapshots/function.js.snap b/packages/commonjs/test/snapshots/function.js.snap
diff --git a/packages/data-uri/package.json b/packages/data-uri/package.json
@@ -22,7 +22,7 @@
     "ci:lint:commits": "commitlint --from=${CIRCLE_BRANCH} --to=${CIRCLE_SHA1}",
     "ci:test": "pnpm run test -- --verbose",
     "lint": "pnpm run lint:js && pnpm run lint:docs && pnpm run lint:package",
-    "lint:docs": "prettier --single-quote --arrow-parens avoid --write README.md",
+    "lint:docs": "prettier --single-quote --arrow-parens avoid --trailing-comma none --write README.md",
     "lint:js": "eslint --fix --cache src test types --ext .js,.ts",
     "lint:package": "prettier --write package.json --plugin=prettier-plugin-package",
     "prebuild": "del-cli dist",

diff --git a/packages/dsv/package.json b/packages/dsv/package.json
@@ -19,7 +19,7 @@
     "ci:lint:commits": "commitlint --from=${CIRCLE_BRANCH} --to=${CIRCLE_SHA1}",
     "ci:test": "pnpm run test -- --verbose",
     "lint": "pnpm run lint:js && pnpm run lint:docs && pnpm run lint:package",
-    "lint:docs": "prettier --single-quote --arrow-parens avoid --write README.md",
+    "lint:docs": "prettier --single-quote --arrow-parens avoid --trailing-comma none --write README.md",
     "lint:js": "eslint --fix --cache src test --ext .js,.ts",
     "lint:package": "prettier --write package.json --plugin=prettier-plugin-package",
     "prebuild": "del-cli dist",

diff --git a/packages/dynamic-import-vars/package.json b/packages/dynamic-import-vars/package.json
@@ -22,7 +22,7 @@
     "ci:lint:commits": "commitlint --from=${CIRCLE_BRANCH} --to=${CIRCLE_SHA1}",
     "ci:test": "pnpm run test -- --verbose",
     "lint": "pnpm run lint:js && pnpm run lint:docs && pnpm run lint:package",
-    "lint:docs": "prettier --single-quote --arrow-parens avoid --write README.md",
+    "lint:docs": "prettier --single-quote --arrow-parens avoid --trailing-comma none --write README.md",
     "lint:js": "eslint --fix --cache src test --ext .js",
     "lint:package": "prettier --write package.json --plugin=prettier-plugin-package",
     "prebuild": "del-cli dist",

diff --git a/packages/html/package.json b/packages/html/package.json
@@ -20,7 +20,7 @@
     "ci:lint:commits": "commitlint --from=${CIRCLE_BRANCH} --to=${CIRCLE_SHA1}",
     "ci:test": "pnpm run test -- --verbose",
     "lint": "pnpm run lint:js && pnpm run lint:docs && pnpm run lint:package",
-    "lint:docs": "prettier --single-quote --arrow-parens avoid --write *.md",
+    "lint:docs": "prettier --single-quote --arrow-parens avoid --trailing-comma none --write *.md",
     "lint:js": "eslint --fix --cache lib test --ext .js,.ts",
     "lint:package": "prettier --write package.json --plugin=prettier-plugin-package",
     "prepublishOnly": "pnpm run lint",

diff --git a/packages/image/package.json b/packages/image/package.json
@@ -22,7 +22,7 @@
     "ci:lint:commits": "commitlint --from=${CIRCLE_BRANCH} --to=${CIRCLE_SHA1}",
     "ci:test": "pnpm run test -- --verbose",
     "lint": "pnpm run lint:js && pnpm run lint:docs && pnpm run lint:package",
-    "lint:docs": "prettier --single-quote --arrow-parens avoid --write *.md",
+    "lint:docs": "prettier --single-quote --arrow-parens avoid --trailing-comma none --write *.md",
     "lint:js": "eslint --fix --cache src test --ext .js,.ts",
     "lint:package": "prettier --write package.json --plugin=prettier-plugin-package",
     "prebuild": "del-cli dist",

diff --git a/packages/inject/package.json b/packages/inject/package.json
@@ -19,7 +19,7 @@
     "ci:lint:commits": "commitlint --from=${CIRCLE_BRANCH} --to=${CIRCLE_SHA1}",
     "ci:test": "pnpm run test -- --verbose && pnpm run test:ts",
     "lint": "pnpm run lint:js && pnpm run lint:docs && pnpm run lint:package",
-    "lint:docs": "prettier --single-quote --arrow-parens avoid --write README.md",
+    "lint:docs": "prettier --single-quote --arrow-parens avoid --trailing-comma none --write README.md",
     "lint:js": "eslint --fix --cache src test --ext .js,.ts",
     "lint:package": "prettier --write package.json --plugin=prettier-plugin-package",
     "prebuild": "del-cli dist",