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

Add example app, gen d.ts files #3

Merged
merged 14 commits into from
Sep 3, 2023
Merged

Add example app, gen d.ts files #3

merged 14 commits into from
Sep 3, 2023

Conversation

AkifumiSato
Copy link
Collaborator

@AkifumiSato AkifumiSato commented Aug 26, 2023

Copy link
Member

@koichik koichik left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

example-nextはなぜnpm? これもpnpmでいいのでは?

【追記】じゃなくてpackage-lock.jsonが残ってるだけ?

apps/example-next/src/app/_components/Providers.tsx Outdated Show resolved Hide resolved
@AkifumiSato
Copy link
Collaborator Author

example-nextはなぜnpm? これもpnpmでいいのでは?

create-next-appした時に--use-pnpmつけ忘れた気がします。
変更すべき対象ばっかチェックしてて見逃しました...
849fdf3

@AkifumiSato
Copy link
Collaborator Author

ローカルでは通るんでCI落ちてるのはturboのキャッシュ周りっぽいですが...
一旦作業時間リミットきそうなので、後でもっかいみてみます。

@AkifumiSato
Copy link
Collaborator Author

AkifumiSato commented Aug 27, 2023

コメントのissueが解決されないとd.tsの生成待たずにturborepoが次の生成に行ってしまうためにd.tsの生成は諦めて戻しました。
また、tsupのconfigファイルを作成しましたが、これはconfigsで共通化しようとしてもtsupがimportを解決できなかったので、これも諦めました。

@koichik
Copy link
Member

koichik commented Aug 28, 2023

tsupまだ時期尚早ですかねー?

@AkifumiSato
Copy link
Collaborator Author

AkifumiSato commented Aug 28, 2023

rollupと比べると冗長な設定もなくプラグインいっぱい入れずに済むから導入ハードルも低くって、個人的には↑くらいならまだ全然許容な範囲の感覚です!
よく考えたらtscする時に生成いいだけな気もしてきたので、それだけ試してみます

@AkifumiSato
Copy link
Collaborator Author

ちょっとやってみたんですが、

  • tsconfigをjestと分けないとxxx.test.d.tsも作られる
  • types/src/xxx.d.tsみたくなってしまう

などあり、結構作業大きくなりそうだったので一旦諦めました。
d.tsの生成はどのくらいの優先度でやりましょう?
この後別PR・リリース諸々整ってから・リリース後・tsupサポート後のどのタイミングでやるか悩んでます。

@koichik
Copy link
Member

koichik commented Sep 1, 2023

d.tsは急がなくてもいいけどそれなしでリリースはありえないのでは
なので他の作業を進めてリリースの準備が完了したらtsupの状況を見て判断でよいかとー

@AkifumiSato
Copy link
Collaborator Author

たまにないパッケージもあるからなしも一瞬考えたんですがやっぱ必須ですよね、、、
tsconfig自体分割してdts生成用コマンド作成しました。

@@ -1,14 +1,24 @@
{
"$schema": "https://turbo.build/schema.json",
"pipeline": {
"dts": {
"dependsOn": ["^dts"],
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

turborepoのdependsOnの説明勘違いしてて^付けてなかったんですが、これが依存関係に依存する処理をturborepoに伝えるものだったので追加しました。
https://turbo.build/repo/docs/reference/configuration#dependson

@AkifumiSato AkifumiSato changed the title Add example app Add example app, gen d.ts files Sep 2, 2023
Copy link
Member

@koichik koichik left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

乙でした!

@AkifumiSato AkifumiSato merged commit 6a443fd into main Sep 3, 2023
1 check passed
@AkifumiSato AkifumiSato deleted the apps branch September 3, 2023 12:39
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.

2 participants