fix: remove unused useContext import in AddTask.js and TaskList.js - #8415
fix: remove unused useContext import in AddTask.js and TaskList.js#8415Jah-yee wants to merge 1 commit into
Conversation
Size changesDetails📦 Next.js Bundle Analysis for react-devThis analysis was generated by the Next.js Bundle Analysis action. 🤖 This PR introduced no changes to the JavaScript bundle! 🙌 |
|
👋 Hi! Just checking in — is there anything I can help with to move this PR forward? Happy to address any feedback! 🙏 |
|
👍 Looks good to merge — clean fix, thanks for contributing! |
|
Good contribution removing unused imports! Just a reminder for maintainer review. Thanks! 🙏 |
|
This PR looks good and is ready to merge! [+1] |
|
Hi @ALL, gentle reminder that this PR is waiting for review. Happy to make any changes — just let me know! 🙏 |
|
Thanks for this contribution! Bumping for visibility. |
|
Would it be possible to merge this PR? Happy to make any adjustments if needed. Thank you for your time! |
|
➿ bump (R337) |
|
bump |
|
🙌 bump |
|
👍 promote |
|
Looks good! Merging soon would be appreciated – thanks for maintaining this project 🙏 |
|
This PR looks great and is ready to merge! 🎉 Thank you for your contribution! |
|
Thank you for this contribution! This PR looks ready to merge. Please consider merging at your earliest convenience. 🌟 |
|
Thank you for maintaining this project! This PR looks ready to merge — a friendly bump to keep it moving. ✅ |
1 similar comment
|
Thank you for maintaining this project! This PR looks ready to merge — a friendly bump to keep it moving. ✅ |
|
Please consider merging this PR. Thank you for maintaining this project! |
1 similar comment
|
Please consider merging this PR. Thank you for maintaining this project! |
|
Thank you for reviewing! This PR removes unused |
|
Just checking in — this PR looks ready to merge. Thanks for the review! 🙏 |
|
Ready for merge! This fix addresses the reported issue. Thanks for reviewing 🙏 |
|
Friendly ping — this documentation typo fix is ready to merge. Thanks for maintaining react.dev! 🙏 |
|
Friendly ping @reactjs/react.dev — typo fix ready to merge. Let me know if anything else is needed! 🙏 |
|
Bumping this PR — still clean and ready to merge. Thanks for the review! 🙏 |
|
This fix looks correct and improves code quality. Ready to merge! 👍 |
|
Still looking good to merge! Thanks for the review 🙏 |
|
Bumping for merge consideration. @reactjs/react.dev |
|
Just checking in on this PR. The fix is minimal and targeted. Would be great to get this merged. Thanks! |
|
Bumping for merge consideration 🚀 |
Good day,
Summary
This PR fixes issue #8186, which reports an unused
useContextimport in two code examples on the useContext reference page.Changes
In the
Scaling up with context and a reducerexample (Example 5 of 5):useContextfrom thereactimport. The component only usesuseStatefor local text state management.useContextfrom thereactimport. The component usesuseTasksfrom context anduseStateis only used in the childTaskcomponent (which correctly imports it).Verification
Both components were reviewed to confirm
useContextwas not called anywhere within the code example boundaries:useState+useTasksDispatch, nouseContextcalluseTasks+useTasksDispatch,useStateis imported and used by the nestedTaskcomponentThank you for your attention. If there are any issues or suggestions, please leave a comment and I will address them promptly.
Warmly,
RoomWithOutRoof