Conversation
## 変更内容 - dojos_earth.json を _data/ ディレクトリに移動 - YAML 変換処理を完全に削除(Jekyll は JSON を直接サポート) - 関連スクリプトのパスを更新 ## 削除したファイル - cache_data_as_yaml.rb(不要な変換スクリプト) - _data/earth.yml(YAML版は不要) ## 更新したファイル - get_data_from_earth.rb: _data/dojos_earth.json に直接出力 - upsert_dojos_geojson.rb: _data/dojos_earth.json から読み込み - earth.json / japan.json: site.data.dojos_earth を使用 - Rakefile: cache_data_as_yaml タスクを削除 - GitHub Actions: パスと不要なタスクを削除 ## メリット - コード削減: 変換スクリプトが不要に - 保守性向上: シンプルな構造で理解しやすい - パフォーマンス: 変換ステップがなくなり高速化 - 一貫性: データ形式が JSON で統一
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
🎯 概要
Jekyll が
_dataディレクトリ内の JSON ファイルを直接サポートすることを活用して、データ管理を大幅に簡素化しました。🔄 主な変更
データファイルの再配置
dojos_earth.jsonを_data/dojos_earth.jsonに移動site.data.dojos_earthとして読み込み不要なファイル・処理の削除
cache_data_as_yaml.rb- YAML変換スクリプト(削除)_data/earth.yml- YAML版データ(削除)cache_data_as_yamlRakeタスク(削除)スクリプトの更新
get_data_from_earth.rb:_data/dojos_earth.jsonに直接出力upsert_dojos_geojson.rb:_data/dojos_earth.jsonから読み込みearth.json/japan.json:site.data.dojos_earthを使用✨ メリット
📊 変更の影響
✅ テスト
earth.json/japan.jsonが正しく生成される(314道場)💡 学び
Jekyll の
_dataディレクトリは YAML だけでなく JSON も直接サポートしているため、変換処理は完全に不要でした。この発見により、コードベースを大幅に簡素化できました。