Skip to content

Conversation

@dependabot
Copy link
Contributor

@dependabot dependabot bot commented on behalf of github Jan 13, 2021

Bumps @bazel/terser from 2.3.1 to 3.0.0.

Release notes

Sourced from @bazel/terser's releases.

3.0.0

Following our usual 6-month cadence for major releases, here is 3.0.0 just in time for your winter vacation! As usual, we've done our best to make breaking changes that improve the long-term health of the project and which are easy to adopt. Give us feedback in the #javascript channel on slack.bazel.build

👏🏻 Thanks to our awesome contributors for this release! 👏🏻

@mistic, @lukasholzer, @duarten, @jbedard, @mrmeku , @opelhoward , @Schibum , @mattinsler , @JiaLiPassion , @fredrikredflag , @KrauseStefan , @comius , @lovepocky , @tylerhou and @OlaviSau

To upgrade:

http_archive(
    name = "build_bazel_rules_nodejs",
    sha256 = "6142e9586162b179fdd570a55e50d1332e7d9c030efd853453438d607569721d",
    urls = ["https://github.com/bazelbuild/rules_nodejs/releases/download/3.0.0/rules_nodejs-3.0.0.tar.gz"],
)

and upgrade your @bazel-scoped npm packages (you'll get an error when installing 2.x version packages with rules_nodejs 3.x)

Finally, read the 📚 Migration instructions from 2.x for breaking changes: https://github.com/bazelbuild/rules_nodejs/wiki#migrating-to-30

BREAKING CHANGES

Stricter installs

For both yarn_install and npm_install we now fail if the lockfile is out-of-date, rather than update it. This assumes you'll use the tooling to manually update the lockfile as needed.

yarn_install now passes --frozen_lockfile by default, but you can use yarn_install(frozen_lockfile = False) to go back to the old behavior.

npm_install now has an attribute npm_command that defaults to ci but you can set it to install to go back to the old behavior.

We also flipped the default for strict_visibility on these two rules. This prevents you adding a Bazel dependency on a library that isn't listed as a dependency in package.json. You can set it back to False if you need the old behavior.

--bazel_patch_module_resolver defaults to false

By default, we no longer patch the require() function, instead you should rely on the linker to make node modules resolvable at the standard location if this breaks you, the quickest fix is to flip the flag back on a nodejs_binary/nodejs_test/npm_package_bin with templated_args = ["--bazel_patch_module_resolver"], see bazelbuild/rules_nodejs#2344 as an example. Another fix is to explicitly use our runfiles helper library, see bazelbuild/rules_nodejs#2341 as an example.

karma and ts_devserver rules moved to new package @bazel/concatjs

packages/karma:package.bzl is gone, in your WORKSPACE replace

load("//packages/karma:package.bzl", "npm_bazel_karma_dependencies")

</tr></table>

... (truncated)

Changelog

Sourced from @bazel/terser's changelog.

3.0.0 (2020-12-22)

For a full list for the breaking changes in 3.0.0 and other notes on migrating, see the Migrating to 3.0.0 wiki page.

Bug Fixes

  • builtin: only pass kwargs to the test, not the .update binary (#2361) (afa095b)

Code Refactoring

  • builtin: remove node_modules attribute from nodejs_binary, nodejs_test & ts_library (c2927af)

BREAKING CHANGES

  • builtin: We removed the node_modules attribute from nodejs_binary, nodejs_test, jasmine_node_test & ts_library.

If you are using the node_modules attribute, you can simply add the target specified there to the data or deps attribute of the rule instead.

For example,

nodejs_test(
    name = "test",
    data = [
        "test.js",
        "@npm//:node_modules",
    ],
    entry_point = "test.js",
)

or

ts_library(
    name = "lib",
    srcs = glob(["*.ts"]),
    tsconfig = ":tsconfig.json",
    deps = ["@npm//:node_modules"],
)

We also dropped support for filegroup based node_modules target and removed node_modules_filegroup from index.bzl.

If you are using this feature for user-managed deps, you must now a js_library target with external_npm_package set to True instead.

For example,

... (truncated)

Commits
  • c2927af refactor(builtin): remove node_modules attribute from nodejs_binary, nodejs_t...
  • f1f700f refactor: explicitly use runfiles helper
  • See full diff in compare view

Dependabot compatibility score

Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting @dependabot rebase.


Dependabot commands and options

You can trigger Dependabot actions by commenting on this PR:

  • @dependabot rebase will rebase this PR
  • @dependabot recreate will recreate this PR, overwriting any edits that have been made to it
  • @dependabot merge will merge this PR after your CI passes on it
  • @dependabot squash and merge will squash and merge this PR after your CI passes on it
  • @dependabot cancel merge will cancel a previously requested merge and block automerging
  • @dependabot reopen will reopen this PR if it is closed
  • @dependabot close will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually
  • @dependabot ignore this major version will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this minor version will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this dependency will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)

@dependabot dependabot bot requested a review from a team January 13, 2021 05:12
@dependabot dependabot bot added area: build & ci Related the build and CI infrastructure of the project action: merge The PR is ready for merge by the caretaker merge safe target: patch This PR is targeted for the next patch release labels Jan 13, 2021
@google-cla google-cla bot added the cla: yes PR author has agreed to Google's Contributor License Agreement label Jan 13, 2021
@mmalerba
Copy link
Contributor

@dependabot rebase

@dependabot dependabot bot force-pushed the dependabot/npm_and_yarn/bazel/terser-3.0.0 branch from 65ec65c to 02def90 Compare January 15, 2021 00:14
@dependabot dependabot bot force-pushed the dependabot/npm_and_yarn/bazel/terser-3.0.0 branch from 02def90 to 3276c1c Compare January 15, 2021 00:46
@mmalerba mmalerba added target: rc This PR is targeted for the next release-candidate and removed target: patch This PR is targeted for the next patch release labels Jan 15, 2021
@mmalerba mmalerba merged commit 95ca505 into master Jan 15, 2021
mmalerba pushed a commit that referenced this pull request Jan 15, 2021
Bumps [@bazel/terser](https://github.com/bazelbuild/rules_nodejs/tree/HEAD/packages/terser) from 2.3.1 to 3.0.0.
- [Release notes](https://github.com/bazelbuild/rules_nodejs/releases)
- [Changelog](https://github.com/bazelbuild/rules_nodejs/blob/stable/CHANGELOG.md)
- [Commits](https://github.com/bazelbuild/rules_nodejs/commits/3.0.0/packages/terser)

Signed-off-by: dependabot[bot] <[email protected]>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
(cherry picked from commit 95ca505)
@dependabot dependabot bot deleted the dependabot/npm_and_yarn/bazel/terser-3.0.0 branch January 15, 2021 18:31
@angular-automatic-lock-bot
Copy link

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

This action has been performed automatically by a bot.

@angular-automatic-lock-bot angular-automatic-lock-bot bot locked and limited conversation to collaborators Feb 15, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

action: merge The PR is ready for merge by the caretaker area: build & ci Related the build and CI infrastructure of the project cla: yes PR author has agreed to Google's Contributor License Agreement target: rc This PR is targeted for the next release-candidate

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant