Skip to content

Commit cba9788

Browse files
authored
fix(replace): prepare for Rollup 3 (#1286)
BREAKING CHANGES: Requires Node 14
1 parent 62f0a13 commit cba9788

File tree

9 files changed

+55
-49
lines changed

9 files changed

+55
-49
lines changed

packages/replace/README.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,7 @@
1313

1414
## Requirements
1515

16-
This plugin requires an [LTS](https://github.com/nodejs/Release) Node version (v8.0.0+) and Rollup v1.20.0+.
16+
This plugin requires an [LTS](https://github.com/nodejs/Release) Node version (v14.0.0+) and Rollup v1.20.0+.
1717

1818
## Install
1919

packages/replace/package.json

Lines changed: 24 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -13,8 +13,16 @@
1313
"author": "Rich Harris <[email protected]>",
1414
"homepage": "https://github.com/rollup/plugins/tree/master/packages/replace#readme",
1515
"bugs": "https://github.com/rollup/plugins/issues",
16-
"main": "dist/rollup-plugin-replace.cjs.js",
17-
"module": "dist/rollup-plugin-replace.es.js",
16+
"main": "dist/cjs/index.js",
17+
"module": "dist/es/index.js",
18+
"exports": {
19+
"types": "./types/index.d.ts",
20+
"import": "./dist/es/index.js",
21+
"default": "./dist/cjs/index.js"
22+
},
23+
"engines": {
24+
"node": ">=14.0.0"
25+
},
1826
"scripts": {
1927
"build": "rollup -c",
2028
"ci:coverage": "nyc pnpm test && nyc report --reporter=text-lcov > coverage.lcov",
@@ -31,6 +39,7 @@
3139
},
3240
"files": [
3341
"dist",
42+
"!dist/**/*.map",
3443
"src",
3544
"types",
3645
"README.md"
@@ -44,21 +53,26 @@
4453
"modules"
4554
],
4655
"peerDependencies": {
47-
"rollup": "^1.20.0 || ^2.0.0"
56+
"rollup": "^1.20.0||^2.0.0||^3.0.0"
57+
},
58+
"peerDependenciesMeta": {
59+
"rollup": {
60+
"optional": true
61+
}
4862
},
4963
"dependencies": {
50-
"@rollup/pluginutils": "^3.1.0",
51-
"magic-string": "^0.25.7"
64+
"@rollup/pluginutils": "^4.2.1",
65+
"magic-string": "^0.26.4"
5266
},
5367
"devDependencies": {
5468
"@rollup/plugin-buble": "^0.21.3",
55-
"del-cli": "^3.0.1",
69+
"del-cli": "^5.0.0",
5670
"locate-character": "^2.0.5",
57-
"rollup": "^2.67.3",
58-
"source-map": "^0.7.3",
59-
"typescript": "^3.9.7"
71+
"rollup": "^3.0.0-7",
72+
"source-map": "^0.7.4",
73+
"typescript": "^4.8.3"
6074
},
61-
"types": "types/index.d.ts",
75+
"types": "./types/index.d.ts",
6276
"ava": {
6377
"workerThreads": false,
6478
"files": [

packages/replace/rollup.config.js

Lines changed: 0 additions & 15 deletions
This file was deleted.

packages/replace/rollup.config.mjs

Lines changed: 13 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,13 @@
1+
import { readFileSync } from 'fs';
2+
3+
import buble from '@rollup/plugin-buble';
4+
5+
import { createConfig } from '../../shared/rollup.config.mjs';
6+
7+
export default {
8+
...createConfig({
9+
pkg: JSON.parse(readFileSync(new URL('./package.json', import.meta.url), 'utf8'))
10+
}),
11+
input: 'src/index.js',
12+
plugins: [buble()]
13+
};

packages/replace/test/form.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@ const { readdirSync, readFileSync } = require('fs');
66

77
const test = require('ava');
88

9-
const replace = require('../dist/rollup-plugin-replace.cjs.js');
9+
const replace = require('..');
1010

1111
const transformContext = {};
1212

packages/replace/test/function.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@ const { readdirSync } = require('fs');
77
const test = require('ava');
88
const { rollup } = require('rollup');
99

10-
const replace = require('../dist/rollup-plugin-replace.cjs.js');
10+
const replace = require('..');
1111

1212
const { execute, getCodeFromBundle } = require('./helpers/util');
1313

packages/replace/test/misc.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@ const { join } = require('path');
55
const test = require('ava');
66
const { rollup } = require('rollup');
77

8-
const replace = require('../dist/rollup-plugin-replace.cjs.js');
8+
const replace = require('..');
99

1010
const { getOutputFromGenerated } = require('./helpers/util');
1111

@@ -63,7 +63,7 @@ test('can be configured with output plugins', async (t) => {
6363
})
6464
);
6565

66-
t.is(code.trim(), 'log("environment", "production");');
66+
t.is(code.trim(), 'log("environment", "production");\n//# sourceMappingURL=main.js.map');
6767
t.truthy(map);
6868
});
6969

packages/replace/test/sourcemaps.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,7 @@ delete global.fetch;
1010
const { SourceMapConsumer } = require('source-map');
1111

1212
global.fetch = fetch;
13-
const replace = require('../dist/rollup-plugin-replace.cjs.js');
13+
const replace = require('..');
1414

1515
const { getOutputFromGenerated } = require('./helpers/util');
1616

pnpm-lock.yaml

Lines changed: 12 additions & 18 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

0 commit comments

Comments
 (0)