Skip to content

Revert "fix: 路線名IPAの接尾辞を英語IPAに置換"#1414

Merged
TinyKitten merged 1 commit into
devfrom
revert-1413-fix/replace-line-name-suffix-with-english-ipa
Mar 5, 2026
Merged

Revert "fix: 路線名IPAの接尾辞を英語IPAに置換"#1414
TinyKitten merged 1 commit into
devfrom
revert-1413-fix/replace-line-name-suffix-with-english-ipa

Conversation

@TinyKitten

@TinyKitten TinyKitten commented Mar 5, 2026

Copy link
Copy Markdown
Member

Reverts #1413

逆効果で「~Line」の発音がおかしくなった

Summary by CodeRabbit

リリースノート

  • Refactor

    • 駅線名サフィックス処理の内部実装を簡素化しました。
    • 不要な変換ロジックを削除し、処理効率を向上させました。
  • Tests

    • 関連するテストを更新し、新しい処理ロジックに対応させました。

@TinyKitten TinyKitten self-assigned this Mar 5, 2026
@coderabbitai

coderabbitai Bot commented Mar 5, 2026

Copy link
Copy Markdown
Contributor

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 88fa5b96-3ace-455b-a85f-c37302c29cbd

📥 Commits

Reviewing files that changed from the base of the PR and between 945a1ba and 7c171c8.

📒 Files selected for processing (2)
  • stationapi/src/domain/ipa.rs
  • stationapi/src/use_case/dto/line.rs

📝 Walkthrough

ウォークスルー

行名の接尾辞処理関数をリファクタリングしました。replace_line_name_suffix関数をstrip_line_name_suffixに名前変更し、戻り値型を(&str, &str)から&strに変更。接尾辞マッピング定数もLINE_NAME_SUFFIX_MAPからLINE_NAME_SUFFIXESに変更し、データ構造を簡略化。関連するテストと使用箇所も更新しました。

変更内容

コホート / ファイル 概要
IPA接尾辞処理の簡素化
stationapi/src/domain/ipa.rs
関数replace_line_name_suffixstrip_line_name_suffixに名前変更し、戻り値型を(&str, &str)から&strに変更。接尾辞データ構造をLINE_NAME_SUFFIX_MAP: &[(&str, &str)]からLINE_NAME_SUFFIXES: &[&str]に変更。IPA置換ロジックを削除し、接尾辞の除去処理に統一。6つのテスト関数名と期待値を更新。
DTO処理の簡素化
stationapi/src/use_case/dto/line.rs
ipa::replace_line_name_suffixの呼び出しをipa::strip_line_name_suffixに更新。name_ipa計算ロジックを簡略化し、直接katakana_to_ipa(strip_line_name_suffix(&line.line_name_k))から導出。

推定レビュー時間

🎯 2 (Simple) | ⏱️ ~10 分

関連する可能性のあるPR

推奨ラベル

fix

ウサギの詩

🐰 接尾辞の処理、シンプルにね
余計な置換は削除して
「除去」に統一、すっきり爽快
戻り値も一筋のstring
関数の意図、クリア明快 ✨

✨ Finishing Touches
  • 📝 Generate docstrings (stacked PR)
  • 📝 Generate docstrings (commit on current branch)
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch revert-1413-fix/replace-line-name-suffix-with-english-ipa

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

@TinyKitten

Copy link
Copy Markdown
Member Author

revertなのでreview待たずにmerge

@TinyKitten TinyKitten merged commit 9d5ac81 into dev Mar 5, 2026
1 check was pending
@TinyKitten TinyKitten deleted the revert-1413-fix/replace-line-name-suffix-with-english-ipa branch March 5, 2026 22:57
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.

1 participant