chore(tests): vitestのimportを明示的にする #2630
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
🚀 プレビュー用ページを作成しました 🚀 更新時点でのコミットハッシュ： |
sevenc-nanashi
approved these changes
Mar 24, 2025
Member
sevenc-nanashi
left a comment
sevenc-nanashi
left a comment
There was a problem hiding this comment.
特に問題なさそう（Ubuntuのe2eが落ちてるのはre-run案件かな）
Member
Author
|
これやっぱりflakyとして２回実行してて片方通ってるのにエラーになってる気がしますね。。。。。。。 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
内容
ESLintの実行環境における挙動の不整合を解消するため、以下の修正を行いました。
これでplaywright系のときは明示的import・vitestのときは省いても良いというバランスだったのが、全部明示的に変わるので一貫性は上がったはず･･･！
（あと間違いも減る･･･････気がする。）
関連 Issue
close #2627
ref #2628
スクリーンショット・動画など
特になし
その他
このプルリクエストは、ローカル環境でpnpm run lint実行時に発生するエラーの原因を調査した結果、ESLintとtsconfigの設定不整合が原因であると判明したため、その対策として行いました。
テスト実行時の環境差異によるエラー発生を防止し、開発環境の一貫性を保つことを目的としています。