Skip to content

Commit 91f890b

Browse files
committed
refactor: migrate_adding_id_to_yamlタスクのコメントを改善
DBの値で上書きする意図とYAMLのカラム順序制御の 両方の目的を明確に説明するコメントに改善。 変更内容: - 冗長なorder設定行を削除(merge!で上書きされるため) - より包括的で理解しやすいコメントを追加 - カラム順序固定の使用例として活用 技術的背景: - `merge!` は既存キーを DB から上書きする - YAML と DB のデータ優先度 / 順序制御に有効 - order は現在手動で更新しているため不要
1 parent 412f0e6 commit 91f890b

File tree

1 file changed

+5
-1
lines changed

1 file changed

+5
-1
lines changed

lib/tasks/dojos.rake

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -106,9 +106,13 @@ http://www.soumu.go.jp/denshijiti/code.html
106106

107107
dojos.map! do |dojo|
108108
d = Dojo.find_by(name: dojo['name'])
109+
110+
# ID など DB の内容で上書きしたいカラムを明示的に指定する。
111+
# YAML の各カラムの先頭に固定させたい場面などにも有効です。
109112
new_dojo = {}
110113
new_dojo['id'] = d.id
111-
new_dojo['order'] = d.order
114+
#new_dojo['order'] = d.order # ID の直後に固定させる場合の例
115+
112116
new_dojo.merge!(dojo)
113117
new_dojo
114118
end

0 commit comments

Comments
 (0)