Skip to content

Conversation

@alan-agius4
Copy link
Contributor

This commit removes the NGCC code and all the related infra setup required to support it.

BREAKING CHANGE: Angular Compatibility Compiler (ngcc) has been removed and as a result Angular View Engine libraries will no longer work

@angular-robot angular-robot bot added detected: breaking change PR contains a commit with a breaking change detected: feature PR contains a feature commit labels Feb 16, 2023
@alan-agius4 alan-agius4 force-pushed the remove-ngcc branch 2 times, most recently from 18eb2bb to e21d72d Compare February 16, 2023 09:33
@alan-agius4 alan-agius4 added the area: compiler Issues related to `ngc`, Angular's template compiler label Feb 16, 2023
@ngbot ngbot bot added this to the Backlog milestone Feb 16, 2023
@angular-robot angular-robot bot removed the detected: feature PR contains a feature commit label Feb 16, 2023
@alan-agius4 alan-agius4 force-pushed the remove-ngcc branch 3 times, most recently from 4634f36 to 3cbfb17 Compare February 16, 2023 11:42
This commit removes the NGCC code and all the related infra setup required to support it.

BREAKING CHANGE: Angular Compatibility Compiler (ngcc) has been removed and as a result Angular View Engine libraries will no longer work
@alan-agius4 alan-agius4 requested a review from alxhub February 16, 2023 13:47
@alan-agius4 alan-agius4 marked this pull request as ready for review February 16, 2023 13:47
@alan-agius4 alan-agius4 added the action: review The PR is still awaiting reviews from at least one requested reviewer label Feb 16, 2023
Copy link
Member

@alxhub alxhub left a comment

Choose a reason for hiding this comment

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

Thanks, ngcc ❤️ but you will not be missed

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
Contributor

@twerske twerske left a comment

Choose a reason for hiding this comment

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

lgtm on the docs changes

Copy link
Member

@alxhub alxhub left a comment

Choose a reason for hiding this comment

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

Reviewed-for: global-approval

@pullapprove pullapprove bot requested a review from jessicajaniuk February 16, 2023 16:52
Copy link
Member

@alxhub alxhub left a comment

Choose a reason for hiding this comment

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

Reviewed-for: global-approvers

Copy link
Member

@pkozlowski-opensource pkozlowski-opensource left a comment

Choose a reason for hiding this comment

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

LGTM

Reviewed-for: public-api

@alan-agius4 alan-agius4 added action: merge The PR is ready for merge by the caretaker and removed action: review The PR is still awaiting reviews from at least one requested reviewer labels Feb 16, 2023
@alan-agius4 alan-agius4 added the target: major This PR is targeted for the next major release label Feb 16, 2023
Copy link
Contributor

@jessicajaniuk jessicajaniuk left a comment

Choose a reason for hiding this comment

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

reviewed-for: public-api, code-ownership

@alxhub
Copy link
Member

alxhub commented Feb 17, 2023

This PR was merged into the repository by commit 48aa96e.

@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 Mar 20, 2023
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: compiler Issues related to `ngc`, Angular's template compiler detected: breaking change PR contains a commit with a breaking change target: major This PR is targeted for the next major release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants