Skip to content

仙台空港アクセス線の廃止済み快速を削除 - #1726

Merged
TinyKitten merged 2 commits into
devfrom
feature/remove-sendai-airport-rapid
Oct 19, 2026
Merged

TinyKitten merged 2 commits into
devfrom
feature/remove-sendai-airport-rapid

Conversation

@TinyKitten

@TinyKitten TinyKitten commented Oct 19, 2026 •

Copy link
Copy Markdown
Member

概要

2026年3月14日のダイヤ改正で普通列車へ変更された、仙台空港アクセス線の快速を削除します。

公式告知: https://www.senat.co.jp/2025/12/12/4483

変更の種類

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

変更内容

  • data/5!station_station_types.csv から廃止済み快速の line_group_cd=958 の9行だけを削除。
  • 普通の line_group_cd=526、名取の両方の駅ID、全駅停車の設定を維持。
  • 他路線と共有する快速の type_cd=102 は削除せず維持。
  • 廃止済み快速の不在、普通の全停車駅、共有種別の存続を確認する回帰テストを3件追加。
  • 回帰テストは修正前に1件失敗し、修正後に3件とも成功することを確認済み。

テスト

  • make fmt が通ること
  • make clippy が通ること(wasm32 ターゲットを含む)
  • make test が通ること

cargo test -p data_validator --test sendai_airport_access: 3件成功。
cargo run -p data_validator: [VALID] No errors reported.。
git diff --check: 成功。
Rust stable 1.99.1、wasm32-unknown-unknown で検証。

make test: 514件成功、既存の手動性能比較テスト1件は既定どおり無視。
make check: native / wasm32 ともに成功。
ローカル検証は canonical CSV を使用。生成データを含む検証は以下のCIで実施しています。

GitHub Actions(コミット 0d69c004b53699f791c2cfaf61443ddf61a3bf85):

Draft PRのためCodeRabbitの自動レビューはスキップされています。

関連Issue

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

Summary by CodeRabbit

  • テスト
    • 仙台空港アクセス線の停車駅や列車種別のデータを検証するテストを追加しました。空港線で使用される種別と、他路線で引き続き利用される種別を確認します。

@github-actions github-actions Bot added feature 要望対応や課題解決 deploy-dev rust and removed feature 要望対応や課題解決 labels Oct 19, 2026
@github-actions

Copy link
Copy Markdown

🚉 停車駅の変更

1 件の削除された停車パターン

● 停車 / ○ 通過 / △ 一部通過 / ◆ 平日停車 / ◇ 休日停車 / ▲ 一部停車

🗑️ 快速 (Rapid) - 仙台空港線 / 東北本線

type_cd=102, line_group_cd=958

変更内容: 9 駅削除

@TinyKitten TinyKitten self-assigned this Oct 19, 2026
@coderabbitai

coderabbitai Bot commented Oct 19, 2026 •

Copy link
Copy Markdown
Contributor

Review in Change Stack →

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration
  • Configuration used: Organization UI
  • Review profile: CHILL
  • Plan: Essentials
  • Run ID: 423eab9a-3c78-4871-8ed3-4e584f932a9f
📥 Commits

Reviewing files that changed from the base of the PR and between 72b0e3a and 0d69c00.

⛔ Files ignored due to path filters: 1
  • data/5!station_station_types.csv is excluded by !**/*.csv
📒 Files selected for processing: 1
  • data_validator/tests/sendai_airport_access.rs

Included review availability: This review used your included allowance. 4 included reviews remain after this review. Your included PR review attempts over the past 7 days set your current allowance at 5 reviews per hour.

📝 Walkthrough

仙台空港アクセス線のCSVデータを検証するRustテストを追加しました。快速グループの不在、ローカルの停車駅と駅ID、他路線での快速タイプの利用を確認します。

Changes: 仙台空港アクセス線のデータ検証

Layer / File(s) Summary
CSVデータの検証
data_validator/tests/sendai_airport_access.rs
CSV読み込み用の共通ヘルパーと3つのテストを追加しました。テストは仙台空港アクセス線の快速グループと駅データ、および他路線での快速タイプを検証します。

Priority: ⬇️ Low

Estimated code review effort: 2 (Simple) | ~8 minutes

Change: Bug fix

Merge Risk: ⚪ Minimal · up to 0d69c

No actionable merge-blocking issue is established for the Sendai Airport Access Line data checks.

Pre-merge checks: Passed 4 · Failed 1

Failed Failed checks: 1 warning

Check name Status Explanation Resolution
Docstring Coverage Warning Docstring coverage is 75.00% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 4 functions across 1 files. Write docstrings for the functions missing them to satisfy the coverage threshold.
Passed Passed checks: 4 passed
Check name Status Explanation
Title check Passed タイトルは、仙台空港アクセス線の廃止済み快速を削除する主要変更を明確に示しています。
Description check Passed 概要、変更の種類、変更内容、テスト、関連Issue、スクリーンショットの各セクションがあります。変更内容と検証結果も具体的です。関連Issueとスクリーンショットは任意のため、説明は要件を満たしています。
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.
✨ Finishing Touches: 💡 1
📝 Generate docstrings: 💡
  • Commit to this branch
  • Create a new PR
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

うさぎはCSVを開き
駅のデータをひとつずつ見る
快速が消えたことを確かめ
ローカルの停車駅を数える
別の路線の快速も確認
月明かりの下、耳を揺らす

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

@TinyKitten
TinyKitten marked this pull request as ready for review October 19, 2026 13:39
@TinyKitten

Copy link
Copy Markdown
Member Author

@coderabbitai review

@github-actions github-actions Bot added the feature 要望対応や課題解決 label Oct 19, 2026
@coderabbitai

coderabbitai Bot commented Oct 19, 2026 •

Copy link
Copy Markdown
Contributor
✅ Action performed

Review finished.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

@TinyKitten
TinyKitten merged commit 423e5ab into dev Oct 19, 2026
3 checks passed
@TinyKitten
TinyKitten deleted the feature/remove-sendai-airport-rapid branch October 19, 2026 13:51
Bloodshoot111 pushed a commit to Aracari-Systems/StationAPI that referenced this pull request Oct 24, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant