Skip to content

社局・路線情報・多言語表記修正#1322

Merged
TinyKitten merged 4 commits into
TrainLCD:devfrom
ts-uc:dev
Sep 13, 2025
Merged

社局・路線情報・多言語表記修正#1322
TinyKitten merged 4 commits into
TrainLCD:devfrom
ts-uc:dev

Conversation

@ts-uc

@ts-uc ts-uc commented Sep 13, 2025

Copy link
Copy Markdown
Contributor
  • 古くなっていたり誤りのある路線情報の修正をしました。
  • 北海道・東北地方の鉄道を中心に、多言語表記の修正および追加を実施しました。
  • 弘南鉄道のナンバリング対応を行いました。
    • 記号の形状が現状のあるものと一致しないため暫定的に REVERSED_ROUND を使用しています。

- 西日本鉄道の読み
- 札幌市交通局のデータ
- 函館市交通局→函館市企業局交通部
- 十和田観光電鉄→十和田観光電鉄線
- 仙台市営地下鉄→仙台地下鉄
- 箱根登山鉄道→小田急箱根
- スカイレール→廃止社局・路線に
- 以下の路線の多言語表記を修正・追加
  - くりはら田園鉄道線
  - フラワー長井線
  - 秋田内陸線
  - 札幌市営地下鉄各線
  - 札幌市電
  - 十和田観光電鉄線
  - 弘南鉄道各線
- 記号の形状は暫定的に REVERSED_ROUND を使用
@coderabbitai

coderabbitai Bot commented Sep 13, 2025

Copy link
Copy Markdown
Contributor

Important

Review skipped

Review was skipped due to path filters

⛔ Files ignored due to path filters (3)
  • data/1!companies.csv is excluded by !**/*.csv
  • data/2!lines.csv is excluded by !**/*.csv
  • data/3!stations.csv is excluded by !**/*.csv

CodeRabbit blocks several paths by default. You can override this behavior by explicitly including those paths in the path filters. For example, including **/dist/** will override the default block on the dist directory, by removing the pattern from both the lists.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

✨ Finishing touches
🧪 Generate unit tests
  • Create PR with unit tests
  • Post copyable unit tests in a comment

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

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

@TinyKitten TinyKitten left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ご対応ありがとうございました。LGTMです。

@TinyKitten TinyKitten merged commit e92e4b5 into TrainLCD:dev Sep 13, 2025
3 checks passed
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.

2 participants