Skip to content

Conversation

@alan-agius4
Copy link
Collaborator

@alan-agius4 alan-agius4 commented Jan 30, 2019

Users outside of Google don't usually need closure annotations.

We should also follow up with ng-packagr so that tsickle is not required as a peerDependency and afterwards, we be able to remove tsickle as a dependency when generating.

//cc @alexeagle and @vikerman

…raries tsconfig

Users outside of Google don't usually need closure annotations.

We should also follow up with ng-packagr to remove tsickle from peerDependency and afterwards, we be able to remove tsickle from the added dependencies when running this schematic
@alan-agius4 alan-agius4 added the target: patch This PR is targeted for the next patch release label Jan 30, 2019
@alan-agius4 alan-agius4 added target: major This PR is targeted for the next major release and removed target: patch This PR is targeted for the next patch release labels Jan 30, 2019
@hansl hansl merged commit 6ae1752 into angular:master Jan 30, 2019
@alan-agius4 alan-agius4 deleted the closure-library branch January 30, 2019 20:47
@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 12, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

target: major This PR is targeted for the next major release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants