Skip to content

バス種別名の「行」接尾辞を削除#1524

Merged
TinyKitten merged 1 commit into
devfrom
feature/drop-bus-type-iki-suffix
May 15, 2026
Merged

バス種別名の「行」接尾辞を削除#1524
TinyKitten merged 1 commit into
devfrom
feature/drop-bus-type-iki-suffix

Conversation

@TinyKitten
Copy link
Copy Markdown
Member

@TinyKitten TinyKitten commented May 15, 2026

概要

バス variant の type_name 生成ロジックから、行き先に付く「行」接尾辞を除去します。

変更の種類

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

変更内容

stationapi/src/import.rs のバス variant 命名ロジック (build_bus_variants 周辺、L2466-L2515) を更新し、以下のとおり「行」接尾辞を削除しました。

ケース 変更前 変更後
循環便(経由地あり) {headsign}行({via}経由・循環) {headsign}({via}経由・循環)
循環便(経由地なし) {headsign}行 (循環) {headsign} (循環)
始発 ≠ headsign {first} → {headsign}行 {first} → {headsign}
headsign のみ {headsign}行 {headsign}
  • 双方向ペア (A ⇔ B) は元々「行」を付けていなかったため変更なし。
  • ローマ字側 (loop_name_r ほか) も元々「行」相当の語を含まないため変更なし。
  • 命名ルールを説明するコメントブロックも実装に合わせて更新。

テスト

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

関連Issue

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

Summary by CodeRabbit

リリースノート

  • スタイル
    • GTFS バス路線の日本語表記形式を更新しました。往路・復路を含む路線名表記、および循環路線・非循環路線の名称から末尾の「行」表記を削除し、より簡潔な形式に統一されます。

Review Change Stack

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@TinyKitten TinyKitten self-assigned this May 15, 2026
@github-actions github-actions Bot added the feature 要望対応や課題解決 label May 15, 2026
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented May 15, 2026

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: a80859f7-2396-4e93-a7c6-62a48274dbd7

📥 Commits

Reviewing files that changed from the base of the PR and between 97dc515 and 7e550aa.

📒 Files selected for processing (1)
  • stationapi/src/import.rs

📝 Walkthrough

Walkthrough

stationapi/src/import.rs のGTFS バス運用変動タイプ名生成ロジックが更新され、日本語フォーマットから「行」表記が削除されました。循環系・非循環系両方の命名ルールが調整されています。

Changes

GTFS バス運用変動命名規則

Layer / File(s) Summary
日本語命名フォーマットの更新
stationapi/src/import.rs
loop_name クロージャでの循環トリップ命名が「経由・循環」「(循環)」形式へ変更され、非循環での first → headsign 表記からも「行」サフィックスが削除されます。

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Possibly related PRs

  • TrainLCD/StationAPI#1518: 同一箇所の loop_name/first → headsign 日本語表記ルールの先行変更であり、本PRはその命名ルールに対する末尾「行」表記の削除調整を実施しています。
  • TrainLCD/StationAPI#1516: GTFS バス運用変動の type_name 生成ロジック改善に関連する変更であり、同じインポートロジック領域でのトリップ変動名命名規則の改善を扱っています。

Suggested labels

deploy-dev, feature

Poem

🐰 ウサギが見つけた、末尾の「行」
循環も直線も、シンプルに統一
「経由・循環」と「始発 → 先」
日本語の名前、今ひとつに整う
バスの旅も、命名も、より美しく ✨

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed プルリクエストのタイトルは、バス種別名から「行」接尾辞を削除するという主要な変更を明確かつ簡潔に要約しており、変更内容と完全に対応している。
Description check ✅ Passed プルリクエストの説明は、テンプレートの必須セクション(概要、変更の種類、変更内容、テスト)をすべてカバーしており、詳細な変更内容表も含まれている。
Docstring Coverage ✅ Passed Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%.
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 docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch feature/drop-bus-type-iki-suffix

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

@github-actions github-actions Bot added deploy-dev and removed feature 要望対応や課題解決 labels May 15, 2026
@TinyKitten TinyKitten merged commit b16e16a into dev May 15, 2026
11 checks passed
@TinyKitten TinyKitten deleted the feature/drop-bus-type-iki-suffix branch May 15, 2026 15:35
@TinyKitten TinyKitten mentioned this pull request May 17, 2026
10 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant