Skip to content

Performance fix: remove globs

Compare
Choose a tag to compare
@alexeagle alexeagle released this 13 Nov 21:26

Builds should now be faster because Bazel doesn't need to scan directories during the analysis phase. Previously we created BUILD files under the Bazel-managed dependencies when using yarn_install that contained glob
https://docs.bazel.build/versions/master/be/functions.html#glob

Note the new included_files option to yarn_install https://bazelbuild.github.io/rules_nodejs/npm_install/npm_install.html#yarn_install which lets you further trim the files needed as inputs to Bazel actions.

BREAKING CHANGE:

  • yarn_install and npm_install no longer generate "lite" targets like @npm//node_modules/glob:glob__lite