Conversation
0809044 to
58d10dc
Compare
miguelpeixe
left a comment
There was a problem hiding this comment.
Thank you!
It's nitpicky, but I'd say this qualifies as chore rather than refactor. No code has changed, only linted. Given the number of impacted files, I think refactor may give the wrong impression when reading the commit history.
|
Hey @adekbadek, good job getting this PR merged! 🎉 Now, the Please check if this PR needs to be included in the "Upcoming Changes" and "Release Notes" doc. If it doesn't, simply remove the label. If it does, please add an entry to our shared document, with screenshots and testing instructions if applicable, then remove the label. Thank you! ❤️ |
|
🎉 This PR is included in version 6.9.0-alpha.1 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
|
🎉 This PR is included in version 6.10.0-alpha.1 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
|
🎉 This PR is included in version 6.10.0 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
All Submissions:
Changes proposed in this Pull Request:
Formats the code using
prettierand ensures formatting is enforced (newspack-scripts@5.6.0).How to test the changes in this Pull Request:
Run
npm run lint:jsand observe no errors.Other information: