Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

jart
Copy link
Contributor

@jart jart commented Jul 9, 2018

This way the Closure Compiler needs to be run 693 fewer times.

$ bazel query 'kind(closure_js_library, deps(//js/closure/...))' | wc -l
802  # <-- before
109  # <-- after

Notes:

  1. These build rules export the world, for the sake of convenience:
    • @io_bazel_rules_closure//closure/library
    • @io_bazel_rules_closure//closure/library:testing
  2. closure_js_library has an implicit dependency on base.js.
  3. rules_closure runs jscomp on each closure_js_library.

This way the Closure Compiler needs to be run 693 fewer times.

    $ bazel query 'kind(closure_js_library, deps(//js/closure/...))' | wc -l
    802  # <-- before
    109  # <-- after

Notes:

1. These build rules export the world, for the sake of convenience:

- @io_bazel_rules_closure//closure/library
- @io_bazel_rules_closure//closure/library:testing

2. closure_js_library has an implicit dependency on base.js.

3. rules_closure runs jscomp on each closure_js_library.
@zongweil zongweil merged commit bd54569 into google:master Jul 10, 2018
@zongweil
Copy link
Contributor

Thanks @jart :).

@jart jart deleted the fewer-deps branch July 10, 2018 09:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants