-
Notifications
You must be signed in to change notification settings - Fork 4
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
Take 2: Upgrade to react-router@7, eslint@9, version bump the rest #102
Conversation
This still has some lingering broken things, as I had to add `"type": "module"` to `package.json` to get it to start. That's got eslint busted
npx codemod remix/2/react-router/upgrade
PR Summary
These changes are aimed at enhancing the overall efficiency, readability, and maintainability of the project by using improved libraries and tools, eliminating redundant code, and refining both server-side and client-side functionalities. |
bold of you to assume |
Had to revert #101 because it wasn't starting up correctly; the built artifact did not include the actual express startup code. This PR copies in the startup script via Docker because of how the React Router v7 "framework mode" build tool functions. #101 has the original motivation for the change.