-
Notifications
You must be signed in to change notification settings - Fork 4.2k
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
Eslint: Add no-unsafe-wp-apis to recommended config #27327
Conversation
Size Change: 0 B Total Size: 1.28 MB ℹ️ View Unchanged
|
Yes, great idea to have it ready to ship when we have more breaking changes in the |
fe5a03b
to
f0358d4
Compare
I merged #27301 so this PR can be updated to contain only its own changes 😄 |
f0358d4
to
4bef436
Compare
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.
I'm approving this PR. It's still to be decided when we will land it though :)
This PR has now merge conflicts to resolve in the changelog file. Once it's done it can be merged. There are a few other PRs with breaking changes that are ready to land. |
4bef436
to
e2a8f4b
Compare
Description
Adds new rule prohibiting unsafe (experimental and unstable) APIs to recommended config. See #27301 which adds the rule.
Types of changes
Breaking change - Adds a new rule to the recommended rules.
Checklist: