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
Fix deprecation warnings for disabled rules #6747
Conversation
🦋 Changeset detectedLatest commit: d3047f4 The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM - hopefully this new release helps users manage deprecations better!
re: the title, wondering if there's a cleaner way to explain what this PR does - e.g. using the word "both". I don't think this is a huge concern though / not merge-blocking.
@mattxwang Thanks for the feedback.
It could be helpful for users if we had a cleaner PR title (also a changelog entry?). 👍🏼 |
Hm, nothing immediately jumps out at me. A handful of suggestions:
or
or
It's certainly a challenging PR title to write 😅 any thoughts? |
@mattxwang Thank you! I adopted your first suggestion. 👍🏼 |
Closes #6744
No, it's self-explanatory.