-
Notifications
You must be signed in to change notification settings - Fork 256
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
chore(deps): Remove history dependency #648
chore(deps): Remove history dependency #648
Conversation
Current dependencies on/for this PR:
This comment was auto-generated by Graphite. |
07e8eb3
to
98a9233
Compare
Ah I guess the tests have that dependency still... WDYT @afonsojramos, should we swap the tests and remove the direct dependency? Or keep the dev dependency and just upgrade the package? |
If there is an alternative to it, I think that we can/should remove it! |
Wow, nice work. It's less code, too! |
This was supposed to be removed as part of the react-router v6 upgrade (which moves history to a direct dependency), but it was clobbered by renovate