mahjongsoul_sniffer icon indicating copy to clipboard operation
mahjongsoul_sniffer copied to clipboard

`GameRecordPlaceholder.uuid` と `GameRecord.uuid` の型アノテーションが実際の型と異なる

Open Apricot-S opened this issue 8 months ago • 0 comments

概要

  • GameRecordPlaceholder.uuid
  • GameRecord.uuid のプロパティの型 (str) と、インスタンス変数の型 (tuple[str]) が異なる

該当箇所 (develop ブランチ) GameRecordPlaceholder.uuid

  • 変数定義 https://github.com/Cryolite/mahjongsoul_sniffer/blob/develop/mahjongsoul_sniffer/game_record.py#L10
  • プロパティ定義 https://github.com/Cryolite/mahjongsoul_sniffer/blob/develop/mahjongsoul_sniffer/game_record.py#L14

GameRecord.uuid

  • 変数定義 https://github.com/Cryolite/mahjongsoul_sniffer/blob/develop/mahjongsoul_sniffer/game_record.py#L1685
  • プロパティ定義 https://github.com/Cryolite/mahjongsoul_sniffer/blob/develop/mahjongsoul_sniffer/game_record.py#L1722

状況

master ブランチでの lint / format の際は、予期せぬ影響が出ないよう型は変更しなかった。

  • https://github.com/Cryolite/mahjongsoul_sniffer/blob/master/mahjongsoul_sniffer/game_record.py#L8
  • https://github.com/Cryolite/mahjongsoul_sniffer/blob/master/mahjongsoul_sniffer/game_record.py#L2045

GameRecord でリポジトリ内を検索したところ、GameRecordPlaceholderGameRecord は他の箇所で使用されていないようである。

原因

変数定義の際誤って末尾にコンマをつけたと推測される

Apricot-S avatar May 28 '24 16:05 Apricot-S