Skip to content

Conversation

devversion
Copy link
Member

Reduces the manual setup for entry-poitns of packages. With
this change, we no longer need to update the dev-app system
config if we add/remove entry-points.

We basically only need to configure entry-points in the
config.bzl file. Though this is not 100% complete yet,
because we still have legacy karma tests which need another
system config. The bazel karma tests do not need this anymore.

@devversion devversion requested review from a team and jelbourn as code owners October 22, 2019 08:51
@googlebot googlebot added the cla: yes PR author has agreed to Google's Contributor License Agreement label Oct 22, 2019
@devversion devversion added pr: merge safe target: major This PR is targeted for the next major release labels Oct 22, 2019
@devversion devversion force-pushed the build/reduce-entry-point-manual-setup-config branch from 00b8e87 to e8a0641 Compare October 24, 2019 08:46
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

@josephperrott josephperrott added pr: lgtm action: merge The PR is ready for merge by the caretaker labels Oct 24, 2019
@mmalerba
Copy link
Contributor

@devversion needs rebase

@mmalerba mmalerba removed the action: merge The PR is ready for merge by the caretaker label Oct 24, 2019
Copy link
Member

@jelbourn jelbourn left a comment

Choose a reason for hiding this comment

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

LGTM

@jelbourn
Copy link
Member

Just needs rebase

@devversion devversion force-pushed the build/reduce-entry-point-manual-setup-config branch from e8a0641 to a3f02a6 Compare October 24, 2019 18:40
Reduces the manual setup for entry-poitns of packages. With
this change, we no longer need to update the dev-app system
config if we add/remove entry-points.

We basically only need to configure entry-points in the
`config.bzl` file. Though this is not 100% complete yet,
because we still have legacy karma tests which need another
system config. The bazel karma tests do not need this anymore.
@devversion devversion force-pushed the build/reduce-entry-point-manual-setup-config branch from a3f02a6 to b8d4a15 Compare October 24, 2019 18:41
@mmalerba mmalerba added action: merge The PR is ready for merge by the caretaker and removed pr: needs rebase labels Oct 24, 2019
@mmalerba mmalerba merged commit 16d2eb8 into angular:master Oct 24, 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 Nov 24, 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 PR author has agreed to Google's Contributor License Agreement target: major This PR is targeted for the next major release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants