Skip to content

refactor: remove any type from ts4.4 upgrade regarding #23671

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Oct 19, 2021

Conversation

zarend
Copy link
Contributor

@zarend zarend commented Sep 30, 2021

Removes the any type used in the previous ts4.4 upgrade PR when
refering to the style property on the element. Creates a new type to
work around the built in CSSStyleDeclaration not including
webkitTapHighlightColor.

@zarend zarend added refactoring This issue is related to a refactoring merge safe labels Sep 30, 2021
@zarend zarend requested a review from crisbeto September 30, 2021 20:22
@google-cla google-cla bot added the cla: yes PR author has agreed to Google's Contributor License Agreement label Sep 30, 2021
@zarend
Copy link
Contributor Author

zarend commented Sep 30, 2021

Pitch for this PR: one or two any's isn't a big deal, but they add up over time.

@zarend zarend added the target: patch This PR is targeted for the next patch release label Oct 1, 2021
Removes the `any` type used in the previous ts4.4 upgrade PR when
refering to the style property on the element. Creates a new type to
work around the built in `CSSStyleDeclaration` not including
`webkitTapHighlightColor`.
Copy link
Member

@crisbeto crisbeto left a comment

Choose a reason for hiding this comment

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

LGTM

@zarend zarend added the action: merge The PR is ready for merge by the caretaker label Oct 4, 2021
@zarend
Copy link
Contributor Author

zarend commented Oct 4, 2021

note to caretaker: this will be safe to merge as long as it compiles internally. There is no need to actually run tests, should be fine as long as it builds.

@zarend zarend added the merge: caretaker note Alert the caretaker performing the merge to check the PR for an out of normal action needed or note label Oct 4, 2021
@mmalerba
Copy link
Contributor

mmalerba commented Oct 4, 2021

merge safe means presubmit does not need to be run, in this case it does need to be run to ensure all projects still compile

@andrewseguin andrewseguin added target: minor This PR is targeted for the next minor release and removed target: patch This PR is targeted for the next patch release labels Oct 19, 2021
@andrewseguin andrewseguin merged commit 62d37ac into angular:master Oct 19, 2021
forsti0506 pushed a commit to forsti0506/components that referenced this pull request Oct 22, 2021
Removes the `any` type used in the previous ts4.4 upgrade PR when
refering to the style property on the element. Creates a new type to
work around the built in `CSSStyleDeclaration` not including
`webkitTapHighlightColor`.
@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 Nov 19, 2021
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: cdk/drag-drop cla: yes PR author has agreed to Google's Contributor License Agreement merge: caretaker note Alert the caretaker performing the merge to check the PR for an out of normal action needed or note refactoring This issue is related to a refactoring target: minor This PR is targeted for the next minor release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants