Skip to content

Conversation

@VincentLanglet
Copy link
Contributor

@VincentLanglet VincentLanglet commented Dec 23, 2024

With the two recent PRs,

AssertCompareToSpecificMethodRector is now fully covered by both

  • AssertInstanceOfComparisonRector
  • AssertCompareOnCountableWithMethodToAssertCountRector

Do we want to

  • Deprecate AssertCompareToSpecificMethodRector in another PR ?
  • At least remove AssertCompareToSpecificMethodRector from the CodeQuality set ? (Since it won't fix anything not covered by others rules)

@TomasVotruba

@VincentLanglet VincentLanglet marked this pull request as ready for review December 23, 2024 12:06
@TomasVotruba
Copy link
Member

At least remove AssertCompareToSpecificMethodRector from the CodeQuality set ?

This looks like best option 👍

Thank you

@TomasVotruba TomasVotruba merged commit cc0de86 into rectorphp:main Dec 23, 2024
6 checks passed
@VincentLanglet
Copy link
Contributor Author

At least remove AssertCompareToSpecificMethodRector from the CodeQuality set ?

This looks like best option 👍

Thank you

#427

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants