Skip to content

Conversation

yasulab
Copy link
Member

@yasulab yasulab commented Sep 10, 2025

概要

PR #79 で実装されたタイムテーブルの重複検知JavaScriptを、HTML ProoferのカスタムテストにHTMLからリファクタリングしました。

変更内容

  • 🗑️ HTMLファイル内のJavaScript検証コードを削除
  • ✅ HTML Prooferのカスタムテストとして重複検知を実装
  • 📅 YAML (_data/time_table.yml) に重複があれば CI 結果として表示

改善点

  1. 本番環境のクリーン化: テスト用のコードを本番HTMLに含めないようにした
  2. CI/CD統合: rake test で自動的に検証
  3. エラー報告: テストが失敗し、重複の詳細が表示される (↓ 以下を参照)

テスト結果サンプル (重複を検知した場合)

$ bundle exec rake test
...
Checking 23 internal links
Checking internal link hashes in 1 file
Ran on 21 files!

For the CustomChecks check, the following failures were found:

* At _site/time-table/index.html:

  タイムテーブルに重複があります [大会議室(セッション)]:
  10:00-13:10: 開会式
  10:10-10:20: 基調講演(宮島さん)

HTML-Proofer found 1 failure!

関連

- HTMLファイル内のJavaScript検証コードを削除
- HTML Prooferのカスタムテストとして重複検知を実装
- 重複があれば CI (bundle exec rake test) でお知らせ

本番環境に手動で確認するテスト用 JavaScript コードを含めずに、
既存の CI/CD で自動的に実行および検証・確認できるようにしました。
@yasulab yasulab changed the title refactor: タイムテーブル重複検知をHTML ProoferカスタムテストへHTMLから移行 refactor: タイムテーブル重複検知を CI 上で動かせるテストに移行 Sep 10, 2025
@yasulab
Copy link
Member Author

yasulab commented Sep 10, 2025

別途出す予定のリファクタリング PR との conflict 解消のため、一旦こちらマージさせてください...!! 🙇💦

@yasulab yasulab merged commit 86e39e9 into add-time-table-page Sep 10, 2025
2 checks passed
@yasulab yasulab deleted the refactor-time-table-validator branch September 10, 2025 06:44
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