search

SoftwareFoundationGroupAtKyotoU / automata

Other
3 stars 10 forks source link

Modularlize user from app.rb #352

Closed skymountain closed 9 years ago

skymountain commented 9 years ago

344

app.rb から user 機能の一部を user.rb に移動しました． 前回同様ここをもっとこうした方がいい，いやいやそんな変更必要ないといった提案も歓迎です．

krtx commented 9 years ago

モジュールのメソッド呼び出しは SomeModule::some_method ではなくSomeModule.some_methodに統一したほうが良いかもしれません https://github.com/bbatsov/ruby-style-guide#double-colons

skymountain commented 9 years ago

返信待ちの部分以外は直しました．

skymountain commented 9 years ago

とりあえず App.@users をキャッシュしないように変更．

skymountain commented 9 years ago

全部修正しました．

krtx commented 9 years ago

:clap:

skymountain commented 9 years ago

:bow: