Skip to content

fix: remove unused useContext import in AddTask.js and TaskList.js - #8415

Open
Jah-yee wants to merge 1 commit into
reactjs:mainfrom
Jah-yee:fix/usecontext-unused-import
Open

fix: remove unused useContext import in AddTask.js and TaskList.js#8415
Jah-yee wants to merge 1 commit into
reactjs:mainfrom
Jah-yee:fix/usecontext-unused-import

Conversation

@Jah-yee

@Jah-yee Jah-yee commented Apr 18, 2026

Copy link
Copy Markdown

Good day,

Summary

This PR fixes issue #8186, which reports an unused useContext import in two code examples on the useContext reference page.

Changes

In the Scaling up with context and a reducer example (Example 5 of 5):

  • **** (code block): Removed unused useContext from the react import. The component only uses useState for local text state management.
  • **** (code block): Removed unused useContext from the react import. The component uses useTasks from context and useState is only used in the child Task component (which correctly imports it).

Verification

Both components were reviewed to confirm useContext was not called anywhere within the code example boundaries:

  • AddTask: Uses useState + useTasksDispatch, no useContext call
  • TaskList: Uses useTasks + useTasksDispatch, useState is imported and used by the nested Task component

Thank you for your attention. If there are any issues or suggestions, please leave a comment and I will address them promptly.

Warmly,
RoomWithOutRoof

@github-actions

Copy link
Copy Markdown

Size changes

Details

📦 Next.js Bundle Analysis for react-dev

This analysis was generated by the Next.js Bundle Analysis action. 🤖

This PR introduced no changes to the JavaScript bundle! 🙌

@Jah-yee

Jah-yee commented Jul 6, 2026

Copy link
Copy Markdown
Author

👋 Hi! Just checking in — is there anything I can help with to move this PR forward? Happy to address any feedback! 🙏

@Jah-yee

Jah-yee commented Jul 6, 2026

Copy link
Copy Markdown
Author

👍 Looks good to merge — clean fix, thanks for contributing!

@Jah-yee

Jah-yee commented Jul 12, 2026

Copy link
Copy Markdown
Author

Good contribution removing unused imports! Just a reminder for maintainer review. Thanks! 🙏

@Jah-yee

Jah-yee commented Jul 13, 2026

Copy link
Copy Markdown
Author

This PR looks good and is ready to merge! [+1]

@Jah-yee

Jah-yee commented Jul 13, 2026

Copy link
Copy Markdown
Author

Hi @ALL, gentle reminder that this PR is waiting for review. Happy to make any changes — just let me know! 🙏

@Jah-yee

Jah-yee commented Jul 13, 2026

Copy link
Copy Markdown
Author

Thanks for this contribution! Bumping for visibility.

@Jah-yee

Jah-yee commented Jul 13, 2026

Copy link
Copy Markdown
Author

Would it be possible to merge this PR? Happy to make any adjustments if needed. Thank you for your time!

@Jah-yee

Jah-yee commented Jul 13, 2026

Copy link
Copy Markdown
Author

➿ bump (R337)

@Jah-yee

Jah-yee commented Jul 13, 2026

Copy link
Copy Markdown
Author

bump

@Jah-yee

Jah-yee commented Jul 13, 2026

Copy link
Copy Markdown
Author

🙌 bump

@Jah-yee

Jah-yee commented Jul 14, 2026

Copy link
Copy Markdown
Author

👍 promote

@Jah-yee

Jah-yee commented Jul 14, 2026

Copy link
Copy Markdown
Author

Looks good! Merging soon would be appreciated – thanks for maintaining this project 🙏

@Jah-yee

Jah-yee commented Jul 14, 2026

Copy link
Copy Markdown
Author

This PR looks great and is ready to merge! 🎉 Thank you for your contribution!

@Jah-yee

Jah-yee commented Jul 14, 2026

Copy link
Copy Markdown
Author

Thank you for this contribution! This PR looks ready to merge. Please consider merging at your earliest convenience. 🌟

@Jah-yee

Jah-yee commented Jul 14, 2026

Copy link
Copy Markdown
Author

Thank you for maintaining this project! This PR looks ready to merge — a friendly bump to keep it moving. ✅

1 similar comment
@Jah-yee

Jah-yee commented Jul 14, 2026

Copy link
Copy Markdown
Author

Thank you for maintaining this project! This PR looks ready to merge — a friendly bump to keep it moving. ✅

@Jah-yee

Jah-yee commented Jul 14, 2026

Copy link
Copy Markdown
Author

Please consider merging this PR. Thank you for maintaining this project!

1 similar comment
@Jah-yee

Jah-yee commented Jul 14, 2026

Copy link
Copy Markdown
Author

Please consider merging this PR. Thank you for maintaining this project!

@Jah-yee

Jah-yee commented Jul 14, 2026

Copy link
Copy Markdown
Author

Thank you for reviewing! This PR removes unused useContext imports — a clean, minimal fix. Happy to address any feedback. 🙂

@Jah-yee

Jah-yee commented Jul 15, 2026

Copy link
Copy Markdown
Author

Just checking in — this PR looks ready to merge. Thanks for the review! 🙏

@Jah-yee

Jah-yee commented Jul 15, 2026

Copy link
Copy Markdown
Author

Ready for merge! This fix addresses the reported issue. Thanks for reviewing 🙏

@Jah-yee

Jah-yee commented Jul 15, 2026

Copy link
Copy Markdown
Author

Friendly ping — this documentation typo fix is ready to merge. Thanks for maintaining react.dev! 🙏

@Jah-yee

Jah-yee commented Jul 15, 2026

Copy link
Copy Markdown
Author

Friendly ping @reactjs/react.dev — typo fix ready to merge. Let me know if anything else is needed! 🙏

@Jah-yee

Jah-yee commented Jul 15, 2026

Copy link
Copy Markdown
Author

Bumping this PR — still clean and ready to merge. Thanks for the review! 🙏

@Jah-yee

Jah-yee commented Jul 15, 2026

Copy link
Copy Markdown
Author

This fix looks correct and improves code quality. Ready to merge! 👍

@Jah-yee

Jah-yee commented Jul 15, 2026

Copy link
Copy Markdown
Author

Still looking good to merge! Thanks for the review 🙏

@Jah-yee

Jah-yee commented Jul 15, 2026

Copy link
Copy Markdown
Author

Bumping for merge consideration. @reactjs/react.dev

@Jah-yee

Jah-yee commented Jul 15, 2026

Copy link
Copy Markdown
Author

Just checking in on this PR. The fix is minimal and targeted. Would be great to get this merged. Thanks!

@Jah-yee

Jah-yee commented Jul 15, 2026

Copy link
Copy Markdown
Author

Bumping for merge consideration 🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant