Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

build: fix //tools/testing:node target deps and ivy build-dist packages #26629

Closed

Conversation

gregmagolan
Copy link
Contributor

Fixes Ivy package-dist generation.

For some reason, artifacts in the legacy package-dist build break the ivy package-dist build. bazel clean in between package-dists generation fixes this.

@gregmagolan
Copy link
Contributor Author

Also includes build fix from #26626

@gregmagolan gregmagolan changed the title Fix ivy build packages build: fix //tools/testing:node target deps and ivy build-dist packages Oct 21, 2018
@IgorMinar IgorMinar added target: patch This PR is targeted for the next patch release action: merge The PR is ready for merge by the caretaker labels Oct 21, 2018
Copy link
Contributor

@IgorMinar IgorMinar left a comment

Choose a reason for hiding this comment

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

I'm approving this just so that we get the upstream unblocked, but I don't like that we have to rely on bazel clean. it seems wrong.

@IgorMinar IgorMinar removed the request for review from alexeagle October 21, 2018 04:02
IgorMinar pushed a commit that referenced this pull request Oct 21, 2018
IgorMinar pushed a commit that referenced this pull request Oct 21, 2018
IgorMinar pushed a commit to IgorMinar/angular that referenced this pull request Oct 21, 2018
@IgorMinar IgorMinar closed this in dc73499 Oct 21, 2018
sculove pushed a commit to sculove/angular that referenced this pull request Nov 2, 2018
sculove pushed a commit to sculove/angular that referenced this pull request Nov 2, 2018
FrederikSchlemmer pushed a commit to FrederikSchlemmer/angular that referenced this pull request Jan 3, 2019
FrederikSchlemmer pushed a commit to FrederikSchlemmer/angular that referenced this pull request Jan 3, 2019
@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 Sep 14, 2019
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 target: patch This PR is targeted for the next patch release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants