X Tutup
Skip to content

Remove angular2/angular2.ts#5844

Closed
tbosch wants to merge 3 commits intoangular:masterfrom
tbosch:remove_angular2
Closed

Remove angular2/angular2.ts#5844
tbosch wants to merge 3 commits intoangular:masterfrom
tbosch:remove_angular2

Conversation

@tbosch
Copy link
Copy Markdown
Contributor

@tbosch tbosch commented Dec 11, 2015

The two top level commits are from #5797 and will be removed after rebasing.

Closes angular#5815

BREAKING CHANGE:

The following symbols are not exported from angular2/angular2 any more:
`UrlResolver`, `AppRootUrl`, `getUrlScheme`, `DEFAULT_PACKAGE_URL_PROVIDER`.
Use imports from `angular2/compiler` instead.
@googlebot
Copy link
Copy Markdown

We found a Contributor License Agreement for you (the sender of this pull request) and all commit authors, but as best as we can tell these commits were authored by someone else. If that's the case, please add them to this pull request and have them confirm that they're okay with these commits being contributed to Google. If we're mistaken and you did author these commits, just reply here to confirm.

@pkozlowski-opensource
Copy link
Copy Markdown
Member

LGTM!

Closes angular#5815

BREAKING CHANGE:

`angular2/angular2` was removed. Use the correct import from one of the barrels. E.g. `angular2/core`, `angular2/platform/browser`,  `angular2/common`, …

Note: This only applies to JavaScript, Dart is not changed.
@tbosch tbosch closed this in 200dc00 Dec 11, 2015
@tbosch tbosch deleted the remove_angular2 branch December 11, 2015 23:02
This was referenced Jan 15, 2016
@angular-automatic-lock-bot
Copy link
Copy Markdown

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 7, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

X Tutup