Skip to content

Commit 8d376dd

Browse files
committed
Remove now-obsolete import cleanup plugin
This was added as part of the "Interaction Tracking" work in facebook#13509 back in 2018. That feature was removed in facebook#20037 in 2020, and this plugin appears to no longer have any effect on the build output.
1 parent 0120bb1 commit 8d376dd

File tree

2 files changed

+0
-33
lines changed

2 files changed

+0
-33
lines changed

scripts/rollup/build.js

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,6 @@ const Stats = require('./stats');
1818
const Sync = require('./sync');
1919
const sizes = require('./plugins/sizes-plugin');
2020
const useForks = require('./plugins/use-forks-plugin');
21-
const stripUnusedImports = require('./plugins/strip-unused-imports');
2221
const dynamicImports = require('./plugins/dynamic-imports');
2322
const disableDomPropertyTreeshake = require('./plugins/disable-domproperty-treeshake');
2423
const Packaging = require('./packaging');
@@ -422,9 +421,6 @@ function getPlugins(
422421
assume_function_wrapper: !isUMDBundle,
423422
renaming: !shouldStayReadable,
424423
}),
425-
// HACK to work around the fact that Rollup isn't removing unused, pure-module imports.
426-
// Note that this plugin must be called after closure applies DCE.
427-
isProduction && stripUnusedImports(pureExternalModules),
428424
// Add the whitespace back if necessary.
429425
shouldStayReadable &&
430426
prettier({

scripts/rollup/plugins/strip-unused-imports.js

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

0 commit comments

Comments
 (0)