Skip to content

Comments

refactor(animations): remove unecessary interface#50662

Closed
JeanMeche wants to merge 1 commit intoangular:mainfrom
JeanMeche:refactor/animations-animation
Closed

refactor(animations): remove unecessary interface#50662
JeanMeche wants to merge 1 commit intoangular:mainfrom
JeanMeche:refactor/animations-animation

Conversation

@JeanMeche
Copy link
Member

Animation is provided by lib.dom.

PR Type

What kind of change does this PR introduce?

  • Refactoring (no functional changes, no api changes)

Does this PR introduce a breaking change?

  • No

@JeanMeche JeanMeche added area: animations action: review The PR is still awaiting reviews from at least one requested reviewer labels Jun 11, 2023
@ngbot ngbot bot modified the milestone: Backlog Jun 11, 2023
@JeanMeche JeanMeche marked this pull request as ready for review June 11, 2023 19:51
@pullapprove pullapprove bot requested a review from jessicajaniuk June 11, 2023 19:51
@JeanMeche JeanMeche force-pushed the refactor/animations-animation branch from 1528d9c to a6c5ab2 Compare June 11, 2023 21:13
`Animation` is provided by `lib.dom`.
@JeanMeche JeanMeche force-pushed the refactor/animations-animation branch from a6c5ab2 to 1e1f7ee Compare June 11, 2023 21:54
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.

LGTM

I think this should be fine in Google, but just to be safe I'll run a TGP.

@jessicajaniuk jessicajaniuk added target: patch This PR is targeted for the next patch release action: global presubmit The PR is in need of a google3 global presubmit and removed action: review The PR is still awaiting reviews from at least one requested reviewer labels Jun 20, 2023
@JeanMeche
Copy link
Member Author

@jessicajaniuk Can we move forward on this ?

@jessicajaniuk
Copy link
Contributor

@JeanMeche Yes, but we still need to run the TGP. That's the hold up.

@jessicajaniuk
Copy link
Contributor

TGP

@jessicajaniuk jessicajaniuk added action: merge The PR is ready for merge by the caretaker merge: caretaker note Alert the caretaker performing the merge to check the PR for an out of normal action needed or note and removed action: global presubmit The PR is in need of a google3 global presubmit labels Aug 2, 2023
@jessicajaniuk
Copy link
Contributor

Caretaker: TGP is green. This is ready to merge.

@dylhunn
Copy link
Contributor

dylhunn commented Aug 7, 2023

This PR was merged into the repository by commit 10bab47.

dylhunn pushed a commit that referenced this pull request Aug 7, 2023
`Animation` is provided by `lib.dom`.

PR Close #50662
@dylhunn dylhunn closed this in 10bab47 Aug 7, 2023
dylhunn pushed a commit that referenced this pull request Aug 7, 2023
`Animation` is provided by `lib.dom`.

PR Close #50662
@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 7, 2023
ChellappanRajan pushed a commit to ChellappanRajan/angular that referenced this pull request Jan 23, 2024
@JeanMeche JeanMeche deleted the refactor/animations-animation branch February 29, 2024 13:40
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: animations merge: caretaker note Alert the caretaker performing the merge to check the PR for an out of normal action needed or note target: patch This PR is targeted for the next patch release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants