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

Migrate all blitz package code into the nextjs fork #391

Closed
2 of 12 tasks
flybayer opened this issue Nov 12, 2021 · 1 comment
Closed
2 of 12 tasks

Migrate all blitz package code into the nextjs fork #391

flybayer opened this issue Nov 12, 2021 · 1 comment
Assignees

Comments

@flybayer
Copy link
Member

flybayer commented Nov 12, 2021

What do you want and why?

In order to streamline nextjs upgrades, we need to move all blitz package code into the nextjs fork. Once that is finished, we can move the nextjs fork to the top level and then merge nextjs upgrades directly into this repo.

  • packages/babel-preset @flybayer [legacy-framework] move blitz babel config into nextjs fork blitz#2903
  • packages/display (some or all already in next package) @beerose [legacy-framework] Move display package to nextjs fork blitz#2989
  • packages/installer
  • packages/generator - wait for [legacy-framework] Subtemplate management blitz#2679
  • packages/cli
    • Migrate all blitz cli commands into next cli, being careful to preserve existing behavior like auto PORT detection, etc
    • packages/server (will basically become obsolete once cli package is moved)
    • inside blitz pkg cli.ts, spawn next binary and pass all args
  • packages/blitz
    • Once this is removed, the nextjs/packages/next package becomes the main entry point for CLI and everything
    • Rename nextjs/packages/next to blitz in package.json name field only
    • Add babel plugin to rewrite imports from next/* to blitz/* for compatibility with people converting nextjs projects to blitz
  • Migrate test/integration tests into nextjs/test/integration/
  • Everything else at top level. Some things like jest config might need removed completely or merged into corresponding config in nextjs/
  • Move nextjs to top level
@flybayer flybayer self-assigned this Nov 12, 2021
@imcodingideas
Copy link

I was thinking about that because with this coming: https://twitter.com/leeerob/status/1458846513451384832

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants