Skip to content

Conversation

yasulab
Copy link
Member

@yasulab yasulab commented Aug 15, 2025

概要

Rails の標準的な命名規則(config/secrets.yml など)に従い、すべてのYAMLファイルの拡張子を .yaml から .yml に統一しました。

背景

Rails では config/secrets.ymlconfig/database.yml など、.yml 拡張子が標準的に使用されています。しかし、本プロジェクトでは db/dojos.yaml など一部のYAMLファイルで .yaml 拡張子が使用されていました。この不統一を解消し、Rails の慣例に従うことで、コードベースの一貫性を向上させます。

変更内容

📁 ファイル名の変更(6ファイル)

  • db/dojos.yamldb/dojos.yml
  • db/dojo_event_services.yamldb/dojo_event_services.yml
  • db/facebook_event_histories.yamldb/facebook_event_histories.yml
  • db/static_event_histories.yamldb/static_event_histories.yml
  • spec/data/invalid_format_of.yamlspec/data/invalid_format_of.yml

📝 コード内の参照箇所を修正

  • app/models/dojo.rb: YAML_PATH定数の更新
  • lib/tasks/dojo_event_services.rake: ファイルパスの更新
  • lib/tasks/dojos_inactivated_at.rake: 3箇所のファイルパスを更新
  • lib/event_service/providers/facebook.rb: YAML_FILE定数の更新
  • lib/event_service/providers/static_yaml.rb: YAML_FILE定数の更新
  • spec/models/dojo_spec.rb: テスト用YAMLファイルパスの更新

📚 ドキュメントの更新

  • README.md: 4箇所の参照を更新
  • doc/how_to_add_dojo.md: 6箇所の参照を更新
  • doc/plan_inactivated_at_column.md: 8箇所の参照を更新
  • doc/how_to_aggregate_stats_and_events.md: 1箇所の参照を更新

テスト結果

  • RSpec: 158 examples, 0 failures
  • Rubocop: 既存の警告のみ(今回の変更に関連なし)

影響範囲

  • データベースへの影響: なし
  • APIへの影響: なし
  • ユーザー向け機能への影響: なし
  • 開発者への影響: 今後は .yml 拡張子で統一されます

確認事項

  • git mv を使用してファイル履歴を保持
  • すべての参照箇所を更新
  • テストが通ることを確認
  • ドキュメントを更新

レビューポイント

  • ファイル名変更の妥当性
  • 参照箇所の漏れがないか
  • Rails の慣例との整合性

Rails の標準的な命名規則(config/secrets.yml など)に従い、
すべてのYAMLファイルの拡張子を .yaml から .yml に変更しました。

- ファイル名の変更(6ファイル):
  - db/dojos.yaml → db/dojos.yml
  - db/dojo_event_services.yaml → db/dojo_event_services.yml
  - db/facebook_event_histories.yaml → db/facebook_event_histories.yml
  - db/static_event_histories.yaml → db/static_event_histories.yml
  - spec/data/invalid_format_of.yaml → spec/data/invalid_format_of.yml

- コード内の参照箇所を修正:
  - app/models/dojo.rb
  - lib/tasks/dojo_event_services.rake
  - lib/tasks/dojos_inactivated_at.rake
  - lib/event_service/providers/facebook.rb
  - lib/event_service/providers/static_yaml.rb
  - spec/models/dojo_spec.rb

- ドキュメントの更新:
  - README.md (4箇所)
  - doc/how_to_add_dojo.md (6箇所)
  - doc/plan_inactivated_at_column.md (8箇所)
  - doc/how_to_aggregate_stats_and_events.md (1箇所)

- RSpec: 158 examples, 0 failures ✅
- Rubocop: 既存の警告のみ(今回の変更に関連なし)
@yasulab yasulab force-pushed the refactor/unify-yaml-extensions-to-yml branch from b226558 to 7e84abd Compare August 15, 2025 14:07
@yasulab yasulab merged commit 9d12b39 into main Aug 15, 2025
5 checks passed
@yasulab yasulab deleted the refactor/unify-yaml-extensions-to-yml branch August 15, 2025 14:09
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