Skip to content

refactor(facade): remove DateWrapper#12027

Merged
chuckjaz merged 1 commit into
angular:masterfrom
alexeagle:rm_datewrapper
Oct 2, 2016
Merged

refactor(facade): remove DateWrapper#12027
chuckjaz merged 1 commit into
angular:masterfrom
alexeagle:rm_datewrapper

Conversation

@alexeagle

Copy link
Copy Markdown
Contributor

Please check if the PR fulfills these requirements

What kind of change does this PR introduce? (check one with "x")

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[ ] Other... Please describe:

What is the current behavior? (You can also link to an open issue here)

What is the new behavior?

Does this PR introduce a breaking change? (check one with "x")

[ ] Yes
[ ] No

If this PR contains a breaking change, please describe the impact and migration path for existing applications: ...

Other information:

@alexeagle alexeagle added the action: review The PR is still awaiting reviews from at least one requested reviewer label Oct 1, 2016
@vicb

vicb commented Oct 1, 2016

Copy link
Copy Markdown
Contributor

Lgtm

@vicb vicb added pr_state: LGTM 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 Oct 1, 2016
@chuckjaz chuckjaz merged commit decd129 into angular:master Oct 2, 2016
chuckjaz pushed a commit to chuckjaz/angular that referenced this pull request Oct 5, 2016
chuckjaz pushed a commit that referenced this pull request Oct 5, 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 9, 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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants