Closed windchime-yk closed 5 months ago
Name | Link |
---|---|
Latest commit | ba1b84a02bcddc7e0fa83220160b40122fbba67f |
Latest deploy log | https://app.netlify.com/sites/js-primer/deploys/6653ffb217c58a00080d9b5c |
Deploy Preview | https://deploy-preview-1737--js-primer.netlify.app |
Preview on mobile | Toggle QR Code...Use your smartphone camera to open QR code link. |
To edit notification comments on pull requests, go to your Netlify site configuration.
関連しそうなので一応 cc @yossydev @jp-knj @SotaYamaguchi
@azu @lacolaco レビューありがとうございました! ご指摘をいただいた部分を修正しましたので、再レビューをお願いします! 🙇
先にコードと依存の変更までやってマージしちゃっていい気がしてきました
@azu
すみません、通知に埋もれていて気づくのが遅れてしまいました!
今見た限りですと前段のutils.parseArgs
が未着手なようなので、先にこちらの変更を進めようと思います!
@azu @lacolaco
修正遅くなりました!
『ユニットテストを記述する』ディレクトリのテスト実装をMochaからnode:test
に変更いたしましたので、レビューをお願いいたします!
また、IssueのTODOを見る限り変更範囲は『ユニットテストを記述する』ディレクトリのみと判断しましたが、読み間違えていればご指摘ください。
ref: #1717 これはテキスト修正のみの対応になります。 実装側の修正については、#1698 が終わり次第着手する予定です。
修正内容としては、表題の通りMochaから
node:test
への変更になります。 変更に際し、以下を考慮しました。describe
/it
も実装されていましたが、Node.js公式ドキュメントではtest
が主流とされていたため、そちらに寄せましたnode --test test/
だとmacOS上でエラーとなるため、デフォルトでtest
フォルダが指定されていることも加味してnode --test
としましたUsers/laco/nodecli/
に統一していますPreview