-
Notifications
You must be signed in to change notification settings - Fork 2.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
feat(replacements): add airbnb-prop-types-to-prop-types-tools #29329
Conversation
Head branch was pushed to by a user without write access
e13e95d
to
84935d1
Compare
oops, sorry, i forgot that you don't want me to force push :-/ i do it everywhere else on github so it's a habit. |
Co-authored-by: HonkingGoose <34918129+HonkingGoose@users.noreply.github.com>
Co-authored-by: HonkingGoose <34918129+HonkingGoose@users.noreply.github.com>
Co-authored-by: HonkingGoose <34918129+HonkingGoose@users.noreply.github.com>
now prettier complains😂 |
This is getting too much like Whack-A-Mole:
Silly idea incoming: maybe we need a Prettier ignore for the ESLint ignore... 🙃 |
🎉 This PR is included in version 37.400.0 🎉 The release is available on: Your semantic-release bot 📦🚀 |
@ljharb congratulations on getting your first PR merged. 🥳 Thank you for sticking with us during the review process! ❤️ I'm happy we finally managed to get our linter ducks in a row. 🦆 🦆 🦆 |
Thanks :-p |
…tebot#29329) Co-authored-by: Rhys Arkins <rhys@arkins.net> Co-authored-by: HonkingGoose <34918129+HonkingGoose@users.noreply.github.com>
Changes
Replace
airbnb-prop-types
with its new name,prop-types-tools
Context
Airbnb gave me the repo, but made me rename the npm package to not have "airbnb" in it.
Documentation (please check one with an [x])
How I've tested my work (please select one)
I have verified these changes via: