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

🗺 Make entry.server and entry.client files optional #4845

Closed
ryanflorence opened this issue Dec 12, 2022 Discussed in #4601 · 5 comments · Fixed by #4600
Closed

🗺 Make entry.server and entry.client files optional #4845

ryanflorence opened this issue Dec 12, 2022 Discussed in #4601 · 5 comments · Fixed by #4600
Assignees
Labels
proposal:remix Proposals from the Remix team.
Milestone

Comments

@ryanflorence
Copy link
Member

ryanflorence commented Dec 12, 2022

Quick thoughts:

We'll need an RFC with more details on this and probably don't want root to be optional.

Discussed in #4601

Originally posted by mcansh November 14, 2022
make entry.server and entry.client files optional would allow for quicker demos and getting started from nothing easier without having to use the remix cli to bootstrap an entire app. just install the dependencies and make a app/routes/index.tsx file and you're done. plus most folks don't touch these files anyways so making them optional and not need to be on disk* is an added bonus (they'll still be on disk, but you know what i mean)

by default it would look for the files and fallback to ones we have stored in @remix-run/dev, which would be copies from https://github.com/remix-run/remix/tree/main/templates/remix

@ryanflorence ryanflorence added the proposal:remix Proposals from the Remix team. label Dec 12, 2022
@ryanflorence ryanflorence moved this to Backlog in Roadmap Dec 12, 2022
@mcansh mcansh changed the title 🗺 Make entry.server, entry.client, and root files optional 🗺 Make entry.server and entry.client files optional Dec 14, 2022
@mcansh mcansh moved this from Backlog to In Progress in Roadmap Jan 10, 2023
@mcansh
Copy link
Collaborator

mcansh commented Jan 10, 2023

PR: #4600

@ryanflorence
Copy link
Member Author

@mjackson to review the PR

@ryanflorence
Copy link
Member Author

j/k get trolled, @jacob-ebey to review

@ryanflorence
Copy link
Member Author

j/k get trolled they'll review it this week.

@mcansh mcansh linked a pull request Feb 1, 2023 that will close this issue
2 tasks
@mcansh mcansh moved this from In Progress to Merged in Roadmap Feb 10, 2023
@github-actions
Copy link
Contributor

🤖 Hello there,

We just published version v0.0.0-nightly-ecba23c-20230211 which involves this issue. If you'd like to take it for a test run please try it out and let us know what you think!

Thanks!

@ryanflorence ryanflorence added this to the v1.14 milestone Feb 16, 2023
@MichaelDeBoey MichaelDeBoey moved this from Merged to Released in Roadmap Mar 7, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
proposal:remix Proposals from the Remix team.
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

4 participants