Skip to content

chore: remove and add external links (#3395)#3396

Merged
KATO-Hiro merged 2 commits intostagingfrom
#3395
Apr 10, 2026
Merged

chore: remove and add external links (#3395)#3396
KATO-Hiro merged 2 commits intostagingfrom
#3395

Conversation

@KATO-Hiro
Copy link
Copy Markdown
Collaborator

@KATO-Hiro KATO-Hiro commented Apr 10, 2026

close #3395

Summary by CodeRabbit

リリースノート

  • 新機能

    • 外部リンクにXおよびmondを追加しました。
  • 機能改善

    • ナビゲーションメニューの投票管理の位置を変更しました。
  • 削除

    • ICPC Japan Problemsの外部リンクを削除しました。

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Apr 10, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro

Run ID: f9594c8b-2d86-4ba6-8ccf-6c09294f0e3c

📥 Commits

Reviewing files that changed from the base of the PR and between 9818ae7 and a190f78.

📒 Files selected for processing (3)
  • src/lib/constants/external-links.ts
  • src/lib/constants/navbar-links.ts
  • src/lib/constants/urls.ts

📝 Walkthrough

Walkthrough

外部リンク定数の更新により、リンク切れしていたICPC Japan Problemsを削除し、XとMondを追加。ナビゲーションメニューの「投票管理」の配置が変更されました。

Changes

Cohort / File(s) Summary
外部リンク定数更新
src/lib/constants/external-links.ts, src/lib/constants/urls.ts
ICPC Japan Problems(リンク切れ)を削除。X(https://x.com/acnovisteps)とMond(https://mond.how/ja/acnovisteps)を新規追加。
ナビゲーション位置調整
src/lib/constants/navbar-links.ts
「投票管理」リンクを「問題のインポート」直後に移動。

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Poem

リンク切れさらば、新たな扉へ 🚪
X と Mond がお出迎え ✨
投票管理、場所も新しく
シンプルな更新、まことに気持ちよし 🎯

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed PRタイトルは外部リンクの削除と追加という主要な変更を正確に要約しており、明確で簡潔です。
Linked Issues check ✅ Passed PR内の変更は#3395の要件をすべて満たしています。ICPC Japan Problemsリンクの削除とmondリンクの追加が実装されています。
Out of Scope Changes check ✅ Passed すべての変更が#3395の要件範囲内です。外部リンク定数の更新と、navbar内の要素の順序調整(chore: fix order)のみです。
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ 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 #3395

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.

Copy link
Copy Markdown
Collaborator Author

@KATO-Hiro KATO-Hiro left a comment

Choose a reason for hiding this comment

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

LGTM

@KATO-Hiro KATO-Hiro merged commit f8b0bdc into staging Apr 10, 2026
3 checks passed
@KATO-Hiro KATO-Hiro deleted the #3395 branch April 10, 2026 22:59
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