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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore: Convert to monorepo #3788

Merged
merged 15 commits into from Feb 4, 2022
Merged

chore: Convert to monorepo #3788

merged 15 commits into from Feb 4, 2022

Conversation

ThangHuuVu
Copy link
Member

@ThangHuuVu ThangHuuVu commented Feb 2, 2022

Reasoning 馃挕

As discussed via #3650, this is the first step toward the final goal: Move all next-auth projects to a monorepo.

  1. Convert this repo into monorepo with app and src using the chosen stack. Rename app and src into for example dev-app and core.

Checklist 馃Б

Setup monorepo toolings

  • Setup yarn workspaces
  • Setup turborepo:
    • Installed turbo, added turbo.json.
    • Integrate with CI
  • Setup changesets:
    • Installed @changeset/cli.
    • Remove semantic-release. Change version of next-auth to 4.1.2.
    • Integrate with CI

Existing commands & packages

  • root's package.json:
    • eslint, typescript, prettier are kept in root
    • Keep version-pr
  • core: is now packages/next-auth.
  • dev-app: is now packages/dev-app.
  • Documentation
  • Ready to be merged

Some highlights:

  • react and react-dom are set to nohoist
  • npm run dev is now yarn turbo run dev. There's no need for npm run dev:setup
  • copy:css and watch:css is moved to next-auth
  • dev-app/tsconfig.json no longer needs path alias

Affected issues 馃師

Partially solves #3650

@ThangHuuVu ThangHuuVu changed the title Convert to monorepo chore: Convert to monorepo Feb 2, 2022
@balazsorban44 balazsorban44 merged commit a72f1b6 into nextauthjs:main Feb 4, 2022
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.

None yet

2 participants