Skip to content

Conversation

@KentaHizume
Copy link
Contributor

@KentaHizume KentaHizume commented Oct 28, 2024

この Pull request で実施したこと

[email protected]から、@5.3.3にダウングレードしました。

これとは別の理由で、
[email protected]に不具合があるようなので、
^を外して"vue-tsc": "2.1.6"にバージョンを固定しています。

モチベーション

現在Maia/Marisのフロントエンドでは[email protected]を使用しています。
一方で、新たに導入した静的解析用のパッケージ
@vue/eslint-config-airbnb-with-typescriptは、typescript-eslintの6系に依存しています。
typescript-eslintは@7.2.0になるまで[email protected]系に対応していないので、
lintの実行時に、typescriptがサポート対象外のバージョンである旨の警告が出力されるからです。

テスト観点

影響

npm run lint の実行時に、下記のような警告が出力されていないこと

SUPPORTED TYPESCRIPT VERSIONS: >=4.3.5 <5.4.0

リグレッション

  • 下記の主要なユースケースが正常に動作し、開発者ツール上に警告が出ないこと
    アイテム一覧表示、買い物かごへ追加、ログイン、注文
  • CIが正常に動作すること
  • CIで実行しない次のnpm scriptsが正常に動作すること
    モック(mock)、E2Eテスト(test:e2)、E2Eテスト(test:e2e:ci)

この Pull request では実施していないこと

AzureADB2C認証サンプルについては別のPRで実施します。

Issues や Discussions 、関連する Web サイトなどへのリンク

なし

@KentaHizume KentaHizume self-assigned this Oct 28, 2024
@KentaHizume KentaHizume added target: Dressca サンプルアプリケーションDresscaに関係がある サンプルAP and removed サンプルAP labels Oct 28, 2024
@KentaHizume KentaHizume added this to the v0.10.2 milestone Oct 28, 2024
@KentaHizume
Copy link
Contributor Author

ローカルで作業した際のnpmのキャッシュが悪さをしている可能性があるので、
まっさらなフォルダにクローンして再度作業する。

@KentaHizume
Copy link
Contributor Author

KentaHizume commented Oct 28, 2024

下記のvue-tscのアップデートのPRで同じ理由で落ちているので、
tsのダウングレード作業ではない別の原因によるものと思われる。

なぜかmaiaのほうは通っている。

@KentaHizume KentaHizume reopened this Oct 28, 2024
@KentaHizume
Copy link
Contributor Author

KentaHizume commented Oct 28, 2024

同じ問題が報告されており、

[email protected]で修正されたことになっているが、
下記のPRに、修正後も発生しているとのコメントがされている。

[email protected]では問題ないとの報告があり、
戻したところ正常に動作した。

@tsuna-can-se tsuna-can-se merged commit 7709387 into main Oct 29, 2024
2 checks passed
@tsuna-can-se tsuna-can-se deleted the feature/[email protected]にダウングレードする branch October 29, 2024 12:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

target: Dressca サンプルアプリケーションDresscaに関係がある

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants