Skip to content

Feature/issue111 - #4

Merged
satoshi-kondo merged 4 commits into
mainfrom
feature/issue111
Nov 21, 2025
Merged

Feature/issue111#4
satoshi-kondo merged 4 commits into
mainfrom
feature/issue111

Conversation

@satoshi-kondo

@satoshi-kondo satoshi-kondo commented Nov 20, 2025

Copy link
Copy Markdown
Contributor

Issue リンク / Link to Issue

https://github.com/airoa-org/hsr_devops/issues/111

やったこと / What I did

  • Wasabi へのアップロード開始/終了を登録するための処理を追加
  • ソースコードをリファクタリング(Template Methodパターンを利用)
  • テストコードをリファクタリング
  • ドキュメントを更新

やってないこと / What I did not

  • 上記以外

チェック観点 / Check Points

  • 大枠問題ないかだけチェックしただければと思います

補足 / Supplement

  • ドキュメントについてはコードの docstring, examples と重複している箇所は削減しています。
  • 一方で、ドキュメントの詳細については、現段階ではチェックする必要はないです。今後も適宜変更を加えていくため。

Comment thread docs/architecture.md
**API Documentation**: See docstrings in [session.py](../src/airoa_lineage/conversion/session.py)
**Usage Example**: [examples/data_conversion.py](../examples/data_conversion.py)

### WasabiUploadSession

@yuya-haruna yuya-haruna Nov 20, 2025

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Wasabiという固有サービス名に限定しないほうが良いのかなとちょっと思いましたがNITSです。

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

おっしゃるとおりで、USB のところもどうしようかなと思いました。
現状では USB や Wasabi の方が直感的でわかりやすいですし、ストレージが変わるタイミングがあれば、Rename するだけでコストは掛からないので、一旦、わかりやすさをとった感じです。

@satoshi-kondo
satoshi-kondo merged commit 58e7007 into main Nov 21, 2025
1 check passed
@satoshi-kondo
satoshi-kondo deleted the feature/issue111 branch December 25, 2025 07:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants