Skip to content

通勤快速のTrainTypeKindをCommuterRapidに揃える#1495

Merged
TinyKitten merged 1 commit into
devfrom
feature/commuter-rapid-kind
May 8, 2026
Merged

通勤快速のTrainTypeKindをCommuterRapidに揃える#1495
TinyKitten merged 1 commit into
devfrom
feature/commuter-rapid-kind

Conversation

@TinyKitten

@TinyKitten TinyKitten commented May 8, 2026

Copy link
Copy Markdown
Member

概要

stationapi/proto を更新して TrainTypeKind.CommuterRapid = 6 を取り込み、data/4!types.csv の通勤快速行を新しい kind 値に揃えました。

変更の種類

  • バグ修正
  • 新機能
  • データの修正・追加
  • リファクタリング
  • ドキュメント
  • CI/CD
  • その他

変更内容

  • stationapi/proto サブモジュールを 887f4d55b54b5f に更新(proto 側の「TrainTypeKindにCommuterRapid = 6追加」コミット docs: add nirsmmy as a contributor #22 を取り込み)
  • data/4!types.csv の通勤快速行 3 件 (type_cd 104 / 165 / 336) の kind6 に設定

テスト

  • cargo fmt --all -- --check が通ること
  • cargo clippy -- -D warnings が通ること
  • cargo testSQLX_OFFLINE=true)が通ること

cargo run -p data_validator も実行: [VALID] No errors reported.

関連Issue

スクリーンショット(任意)

Summary by CodeRabbit

  • 保守作業
    • 内部の依存関係を更新しました。

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@TinyKitten TinyKitten self-assigned this May 8, 2026
@github-actions github-actions Bot added feature 要望対応や課題解決 deploy-dev labels May 8, 2026
@coderabbitai

coderabbitai Bot commented May 8, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack
No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 75a765da-4df1-4a2f-852b-934a13d03bc9

📥 Commits

Reviewing files that changed from the base of the PR and between 4376df7 and b90c8f7.

⛔ Files ignored due to path filters (1)
  • data/4!types.csv is excluded by !**/*.csv
📒 Files selected for processing (1)
  • stationapi/proto

📝 Walkthrough

Walkthrough

このプルリクエストは、stationapi/protoGitサブモジュールの参照を新しいコミットハッシュに更新します。追跡対象コミットが887f4d...から5b54b5...に変更されています。

Changes

サブモジュール参照更新

Layer / File(s) Summary
Submodule Reference
stationapi/proto
stationapi/protoサブモジュールの参照がコミットハッシュ887f4d...から5b54b5...に更新されます。

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

🐰 プロトコル更新、新しいコミット
サブモジュール指す先が変わりて
バージョン進む、ステーションAPI
定義は新しく、未来へ進む
小さき変更、大きな意味よ

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed PRタイトルは「通勤快速のTrainTypeKindをCommuterRapidに揃える」で、変更内容と直結しており、TrainTypeKindを新しい値に揃えるという主な変更を明確に表現しています。
Description check ✅ Passed PR説明は必須項目をすべてカバーしており、概要、変更の種類、変更内容、テスト結果が完全に記載されています。テンプレートの構造に従い、実行されたテストもすべて記載されています。
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch feature/commuter-rapid-kind

Comment @coderabbitai help to get the list of available commands and usage tips.

@TinyKitten TinyKitten merged commit cec2baf into dev May 8, 2026
14 checks passed
@TinyKitten TinyKitten deleted the feature/commuter-rapid-kind branch May 8, 2026 13:22
@TinyKitten TinyKitten mentioned this pull request May 13, 2026
10 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

deploy-dev feature 要望対応や課題解決

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant