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

refactor: simplify server entrypoint #373

Closed
wants to merge 30 commits into from

Conversation

hi-ogawa
Copy link
Owner

@hi-ogawa hi-ogawa commented May 6, 2023

hattip ideas are mostly from https://github.com/hi-ogawa/vite-fullstack-example/blob/92649f99b041820ec86650c99cfcd49a72e79f71/src/server/hattip.ts#L10-L11

I don't feel it gives any significant advantage...
Btw, cannot use hattip for dev since we need to serve static locally, but hattip doesn't have such middleware yet.

@hi-ogawa hi-ogawa force-pushed the refactor-server-entrypoint branch from 051b69e to 3f8cf80 Compare May 26, 2023 08:39
@hi-ogawa hi-ogawa marked this pull request as ready for review May 26, 2023 11:20
@hi-ogawa hi-ogawa changed the title refactor: server entrypoint refactor: simplify server entrypoint May 26, 2023
@hi-ogawa hi-ogawa marked this pull request as draft May 26, 2023 23:57
@hi-ogawa
Copy link
Owner Author

If I need to do this, I would rather migrate to vite-plugin-ssr entirely...

@hi-ogawa hi-ogawa marked this pull request as ready for review May 28, 2023 11:38
@hi-ogawa
Copy link
Owner Author

hi-ogawa commented Jun 1, 2023

It feels like patching @remix-run/dev is easier than trying to setup own remix watch workflow.
Thus closing this in favor of

@hi-ogawa hi-ogawa closed this Jun 1, 2023
@hi-ogawa hi-ogawa deleted the refactor-server-entrypoint branch June 1, 2023 08:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

chore: dev workflow with remix watch
1 participant