-
Notifications
You must be signed in to change notification settings - Fork 1
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 core implementation and testing #1
Conversation
パッケージの構成ですが、こんな風にしたいと思ってます
なのでNext.js依存のモジュールは |
そうすると |
この程度の規模のライブラリなら |
"npm-run-all"要るかな? pnpmかTurborepoか分からないけどそれ相当の機能を持ってる気がするけど |
|
|
いくつか修正しました。 |
どうせならこれで始めたほうがいいのでは? リポジトリは作り直しても構わないので |
試しに作り直してみようとやってみてたんですが、ディレクトリ構成をこのPRの形に戻したりbuild周り整備する方がちょっと面倒そうな気がしてきました。 |
turborepo入れてみました。 |
乙です! |
|
んー、共有するわけでもなく一つの配布単位を細切れにするメリットを感じないですね むしろディレクトリを減らしたいくらい |
はい、僕も分けたいわけじゃないので大丈夫です! |
設定系はキャッシュというより共有という観点で分けたい (一カ所にまとめたい) ってのはありますね |
直下に親を置いて各packageでextendsして、って思ってましたがtsconfig/eslint-config/jestもprivate packageで作りますか? |
次のステップでやってみてください! |
元から僕の環境ではignoreされてたから気づきませんでした。 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ひとまずここまででマージしてもいいと思います!
src
配下はほとんど持ってきただけで、パスの修正とindex.ts
の作成だけ行いました