Skip to content
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

fix(react): remove outdated packages #16786

Merged

Conversation

ndcunningham
Copy link
Contributor

The features should be available by default or they are available via a different library.
Example
react-test-renderer -> @testing-library

closed #11882

@vercel
Copy link

vercel bot commented May 4, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
nx-dev ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 5, 2023 5:34pm

@xiongemi
Copy link
Collaborator

xiongemi commented May 5, 2023

do we need a migration script to remove these libraries in package.json?

@ndcunningham ndcunningham force-pushed the fix/remove-nextjs-outdated-deps branch from 7aeed36 to 07a6f76 Compare May 5, 2023 15:12
@ndcunningham ndcunningham force-pushed the fix/remove-nextjs-outdated-deps branch 2 times, most recently from 877e446 to 07d438d Compare May 5, 2023 16:47
@ndcunningham ndcunningham force-pushed the fix/remove-nextjs-outdated-deps branch from 07d438d to 7a0dd78 Compare May 5, 2023 17:31
@ndcunningham ndcunningham changed the title fix(nextjs): remove outdated packages fix(react, web): remove outdated packages May 5, 2023
@AgentEnder AgentEnder changed the title fix(react, web): remove outdated packages fix(react): remove outdated packages May 5, 2023
@AgentEnder AgentEnder merged commit 4750a1d into nrwl:master May 5, 2023
@github-actions
Copy link

This pull request has already been merged/closed. If you experience issues related to these changes, please open a new issue referencing this pull request.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators May 11, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Potentially redundant dependencies in Next.js template
4 participants