misskey
misskey copied to clipboard
misskey-dev
Use turborepo
https://github.com/misskey-dev/misskey/issues/9575 https://github.com/misskey-dev/misskey/issues/9955
What
gulpから、turborepoに書き換え
Why
-
gulp関連は、メンテナンス面や脆弱性対応の面で難がある状態になっているため独自のビルドスクリプトに書き換え
-
turborepoは、インクリメンタルビルドが使えて速い
Additional info (optional)
- pnpm format gulpタスクにないため、削除
- pnpm start typeミスがあったため、
./built/boot/index.jsに合わせる
Codecov Report
Merging #9991 (aadfcda) into develop (f5dfb64) will decrease coverage by
0.09%. The diff coverage isn/a.
@@ Coverage Diff @@
## develop #9991 +/- ##
===========================================
- Coverage 77.47% 77.39% -0.09%
===========================================
Files 909 907 -2
Lines 91516 91054 -462
Branches 6890 6832 -58
===========================================
- Hits 70904 70472 -432
+ Misses 20612 20582 -30
see 3 files with indirect coverage changes
![codecov[bot] avatar](https://avatars.githubusercontent.com/in/254?v=4 "Published by a pub.dev verified publisher"){kind=small}
とりあえず…文字列の連結はテンプレートリテラルを使って欲しいです
そんなこと言われても元のコードが+連結だし…感
そもそもPathの連結はpath.resolveの方がいいのではと
とりあえずconflictを潰した上で、一通り動作確認して問題なければとりあえずマージし、問題が起きたら適宜直す方針でどうでしょう？こういうのって仮に問題があったとして、目だけで見ても気づけないと思う。
conflict潰しておきます
本流にマージするのは慎重にやりたいね。失敗すると手が止まるので当然差し戻したくなると思うし、とはいえmonorepoにpizzaxなども含める予定なことを踏まえると、早めに手を打ちたい（既にmisskey-jsの合流やstorybook導入などの差分でちょっと怖くなってる）
開発者向けの変更点としては build コマンドが build:pre になったことくらいかしら？
はい、その通りです。開発者向けの変更点として、 misskey-js の buildコ マンドが build:pre に変更されたことが挙げられます。
@Neko7sora この問題 https://github.com/misskey-dev/misskey/issues/9955#issuecomment-1508291231 のワークアラウンド（または実行要件）ご教示いただければ幸いです
- やってることはシンプルだしあまりキャッシュできることやってるわけじゃないし生でよくね (https://github.com/misskey-dev/misskey/issues/9575#issuecomment-1383048357)
- #10894 でビルドフローいじりたいからこれ入れると邪魔になりそう
という気持ちがある
テストが落ちている理由
1
！Test (production install and build) は、Packagesが正常にインストールされない
API report (misskey.js) / report (pull_request) などは、正常にインストールされる
2
！Storybook / build は、変更がテストに反映されていない
3
！Test (frontend) / e2e (18.x, chrome) 不明