Skip to content
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

refactor(multiple): remove coercion members #24055

Merged
merged 1 commit into from
Dec 14, 2021

Conversation

crisbeto
Copy link
Member

@crisbeto crisbeto commented Dec 4, 2021

  • Replaces all usages of the ngAcceptInputType members with type declarations directly on the setters.
  • Replaces the lint rule that was checking that the correct coercion members were added with a rule that doesn't allow new coercion members to be added.

@google-cla google-cla bot added the cla: yes PR author has agreed to Google's Contributor License Agreement label Dec 4, 2021
@crisbeto crisbeto force-pushed the coercion-members-again branch 3 times, most recently from ef519a1 to a5c2e4f Compare December 4, 2021 09:14
@crisbeto crisbeto marked this pull request as ready for review December 4, 2021 09:31
@crisbeto crisbeto added P2 The issue is important to a large percentage of users, with a workaround target: patch This PR is targeted for the next patch release labels Dec 4, 2021
* Replaces all usages of the `ngAcceptInputType` members with type declarations directly on the setters.
* Replaces the lint rule that was checking that the correct coercion members were added with a rule that doesn't allow new coercion members to be added.
@crisbeto crisbeto added the action: merge The PR is ready for merge by the caretaker label Dec 6, 2021
@amysorto amysorto merged commit 24180d1 into angular:master Dec 14, 2021
amysorto pushed a commit that referenced this pull request Dec 14, 2021
* Replaces all usages of the `ngAcceptInputType` members with type declarations directly on the setters.
* Replaces the lint rule that was checking that the correct coercion members were added with a rule that doesn't allow new coercion members to be added.

(cherry picked from commit 24180d1)
@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 Jan 14, 2022
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 PR author has agreed to Google's Contributor License Agreement P2 The issue is important to a large percentage of users, with a workaround target: patch This PR is targeted for the next patch release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants