Closed Kanahiro closed 4 months ago
GTFSデータ処理のためのPythonモジュールが大幅に更新されました。主な変更点は、データ構造へのアクセス方法の変更と関数ベースの設計からクラスベースの設計への移行です。これにより、コードの可読性と保守性が向上しています。
ファイル | 変更概要 |
---|---|
.../aggregate.py , .../parse.py |
GTFS クラスインスタンスの使用とデータアクセス方法の変更 |
.../gtfs.py |
GTFS 関数から GTFS クラスへのリファクタリング、GTFSFactory 関数の導入 |
tests/... |
GTFSFactory の使用への更新、フィクスチャの設定変更 |
🐰✨ ぴょんぴょん跳ねる、コードの変更 新しいクラスで、データを綺麗に パーサー輝く、未来へと うさぎも喜ぶ、GTFSの春 🌸🚌💻
codecov/patch success (17)
gtfs_parser/gtfs.py: [warning] 65-65: gtfs_parser/gtfs.py#L65 Added line #L65 was not covered by tests --- gtfs_parser/gtfs.py: [warning] 74-74: gtfs_parser/gtfs.py#L74 Added line #L74 was not covered by tests
gtfs_parser/gtfs.py (2)
`16-37`: データクラス `GTFS` の実装を確認しました。属性の型指定が適切に行われており、オプショナルな属性には `Optional` 型が使用されています。これにより、型安全性が向上しています。 --- Line range hint `44-127`: `GTFSFactory` 関数のレビューを行いました。関数は適切にファイルを読み込み、データを解析し、エラー状態に対して例外を発生させます。また、データ型のキャストやデフォルト値の設定も行っています。ただし、一部の行がテストによってカバーされていないため、関数の信頼性を確保するためにこれらの行のテストを追加することをお勧めします。 ```diff # テストカバレッジが不足している行に対するテストケースを追加することを提案します。 ```
Attention: Patch coverage is 80.00000%
with 18 lines
in your changes are missing coverage. Please review.
Project coverage is 82.50%. Comparing base (
4f6c890
) to head (c7ac530
). Report is 1 commits behind head on main.
Files | Patch % | Lines |
---|---|---|
gtfs_parser/aggregate.py | 48.27% | 15 Missing :warning: |
gtfs_parser/gtfs.py | 92.85% | 2 Missing :warning: |
gtfs_parser/parse.py | 90.00% | 1 Missing :warning: |
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Close #0
Description(変更内容)
Manual Testing(手動テスト)
Not required / 不要