bin/setup に必須ではないコマンドはコメントアウトして１箇所にまとめた

Author: yasulab

bin/setup

@@ -1,6 +1,5 @@
 #!/usr/bin/env ruby
 require "fileutils"
-require "active_support/time"
 
 APP_ROOT = File.expand_path("..", __dir__)
 APP_NAME = "coderdojo-jp"
@@ -31,6 +30,9 @@ FileUtils.chdir APP_ROOT do
   system! 'bin/rails dojo_event_services:upsert'
   system! "bin/rails statistics:aggregation[-,-,static_yaml]"
 
+  # TODO: This would be better to move outside bin/setup. 'static_yaml' is enough fine.
+  #
+  #require "active_support/time"
   #today = Time.now.in_time_zone('Tokyo').to_date
   #from  = (today - 90).strftime('%Y%m')
   #to    = today.prev_month.strftime('%Y%m')
@@ -42,7 +44,13 @@ FileUtils.chdir APP_ROOT do
   #  system! "bin/rails statistics:aggregation[#{from},#{to},facebook]"
   #  system! 'bin/rails upcoming_events:aggregation[connpass]'
   #else
-  #  puts event_services_message
+  #  puts <<~MESSAGE
+  #    環境変数 DOORKEEPER_API_TOKEN と FIXIE_URL が設定されていないため、
+  #    Doorkeeper API や connpass API を使ったイベント情報の取得をスキップします。
+  #
+  #    なお API 経由でイベント情報を取得しなくても、rails server は実行可能です。
+  #    もし API 経由でイベント情報を取得したい場合は開発チームにご連絡ください。
+  #  MESSAGE
   #end
 
   puts "\n== Removing old logs and tempfiles =="
@@ -55,14 +63,3 @@ FileUtils.chdir APP_ROOT do
   # system "ln -nfs #{APP_ROOT} ~/.puma-dev/#{APP_NAME}"
   # system "curl -Is https://#{APP_NAME}.test/up | head -n 1"
 end
-
-def event_services_message
-  <<~MESSAGE
-
-  環境変数 DOORKEEPER_API_TOKEN と FIXIE_URL が設定されていないため、
-  Doorkeeper API や connpass API を使ったイベント情報の取得をスキップします。
-
-  なお API 経由でイベント情報を取得しなくても、rails server は実行可能です。
-  もし API 経由でイベント情報を取得したい場合は開発チームにご連絡ください。
-  MESSAGE
-end