Skip to content

Commit

Permalink
Remove now-obsolete import cleanup plugin
Browse files Browse the repository at this point in the history
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.
  • Loading branch information
markerikson committed Mar 20, 2023
1 parent 4dd1dc7 commit 473c59b
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 33 deletions.
4 changes: 0 additions & 4 deletions scripts/rollup/build.js
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,6 @@ const Stats = require('./stats');
const Sync = require('./sync');
const sizes = require('./plugins/sizes-plugin');
const useForks = require('./plugins/use-forks-plugin');
const stripUnusedImports = require('./plugins/strip-unused-imports');
const dynamicImports = require('./plugins/dynamic-imports');
const Packaging = require('./packaging');
const {asyncRimRaf} = require('./utils');
Expand Down Expand Up @@ -430,9 +429,6 @@ function getPlugins(
assume_function_wrapper: !isUMDBundle,
renaming: !shouldStayReadable,
}),
// HACK to work around the fact that Rollup isn't removing unused, pure-module imports.
// Note that this plugin must be called after closure applies DCE.
isProduction && stripUnusedImports(pureExternalModules),
// Add the whitespace back if necessary.
shouldStayReadable &&
prettier({
Expand Down
29 changes: 0 additions & 29 deletions scripts/rollup/plugins/strip-unused-imports.js

This file was deleted.

0 comments on commit 473c59b

Please sign in to comment.