-
Notifications
You must be signed in to change notification settings - Fork 315
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: add swapfile #232
fix: add swapfile #232
Conversation
I understand the need for the swapfile, but can this be implemented using the |
@rubys if that's the case, I guess we can simplify |
@MichaelDeBoey yes that is correct. And dockerfile-node already has support for this: fly-apps/dockerfile-node@d489e3a ... and if you check the test cases you will see that this already is enabled and tested for indie and blues stacks. Any chance remix could consider switching to dockerfile node? |
@rubys if we can support everything that's happening right now in the Dockerfile, I'd be happy to accept a PR that changes docs and CI & removes the current version of Dockerfile, |
I believe we can support everything. If we discover something that needs to be added that can be done quickly, so I'm not worried about that. The doc changes should be pretty straightforward. Much of the current README can be replaced by The CI will likely take several iterations to get right. I've already got a test that builds the Dockerfile. Example: https://github.com/fly-apps/dockerfile-node/actions/runs/5933428308/job/16088842745 If looks like your existing test deploys the app on fly.io. It doesn't create the application, so it presumes that the app is already present? So the next step is presumably to figure out what test is needed, subtract out what is already covered, and then write a test for what remains. Whether that test or tests resides in a remix repository or a fly apps repository doesn't matter to me. |
Just like @kwigley's remix-run/blues-stack#183
Fixes #200