X Tutup
Skip to content

fix(bundles): rename external-dependencies to angular-polyfills#5716

Closed
pkozlowski-opensource wants to merge 1 commit intoangular:masterfrom
pkozlowski-opensource:external_deps_rename
Closed

fix(bundles): rename external-dependencies to angular-polyfills#5716
pkozlowski-opensource wants to merge 1 commit intoangular:masterfrom
pkozlowski-opensource:external_deps_rename

Conversation

@pkozlowski-opensource
Copy link
Copy Markdown
Member

Closes #5714

BREAKING CHANGE:

The external-dependencies.js bundle was removed.
Use angular-polyfills.js instead.

Closes angular#5714

BREAKING CHANGE:

The `external-dependencies.js` bundle was removed.
Use `angular-polyfills.js` instead.
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

ok, sorry. it should really be angular2-polyfills.js to align it with angular2.* files. Sorry! I didn't see it until I looked at the diff.

@IgorMinar IgorMinar added pr_state: LGTM action: cleanup The PR is in need of cleanup, either due to needing a rebase or in response to comments from reviews labels Dec 8, 2015
@IgorMinar
Copy link
Copy Markdown
Contributor

@jelbourn can you please rename angular-polyfills.js to angular2-polyfills.js and merge this in? thanks!

@IgorMinar IgorMinar added the action: merge The PR is ready for merge by the caretaker label Dec 8, 2015
@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

action: cleanup The PR is in need of cleanup, either due to needing a rebase or in response to comments from reviews action: merge The PR is ready for merge by the caretaker cla: yes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Rename the external-dependencies.js bundle to....

4 participants

X Tutup