Skip to content

Conversation

devversion
Copy link
Member

Updates to the latest version of rules_nodejs that contains
various fixes for Windows. The linker has been preventing windows
developers from running tests reliably. This has been fixed with:

bazel-contrib/rules_nodejs@7cf7d73

Updates to the latest version of `rules_nodejs` that contains
various fixes for Windows. The linker has been preventing windows
developers from running tests reliably. This has been fixed with:

bazel-contrib/rules_nodejs@7cf7d73
@google-cla google-cla bot added the cla: yes PR author has agreed to Google's Contributor License Agreement label May 13, 2021
@devversion devversion added merge safe target: patch This PR is targeted for the next patch release labels May 13, 2021
@devversion devversion marked this pull request as ready for review May 13, 2021 18:17
@devversion devversion requested a review from a team as a code owner May 13, 2021 18:17
Copy link
Member

@josephperrott josephperrott left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Member

@crisbeto crisbeto left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@devversion devversion added action: merge The PR is ready for merge by the caretaker P2 The issue is important to a large percentage of users, with a workaround labels May 14, 2021
@devversion
Copy link
Member Author

I'm adding P2 because this is merge safe and currently makes developing on Windows really difficult.

@andrewseguin andrewseguin added target: minor This PR is targeted for the next minor release and removed target: patch This PR is targeted for the next patch release labels May 19, 2021
@andrewseguin andrewseguin merged commit 8bbe9bb into angular:master May 19, 2021
andrewseguin pushed a commit that referenced this pull request May 19, 2021
Updates to the latest version of `rules_nodejs` that contains
various fixes for Windows. The linker has been preventing windows
developers from running tests reliably. This has been fixed with:

bazel-contrib/rules_nodejs@7cf7d73
@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 Jun 19, 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 cla: yes PR author has agreed to Google's Contributor License Agreement P2 The issue is important to a large percentage of users, with a workaround target: minor This PR is targeted for the next minor release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants