Prefer 'each' instead of 'for in' #1137

merged 1 commit into from May 19, 2011


None yet
2 participants

guilleiguaran commented May 19, 2011

Little change in activemodel/lib/active_model/observing.rb

@josevalim josevalim added a commit that referenced this pull request May 19, 2011

@josevalim josevalim Merge pull request #1137 from guilleiguaran/each_syntax_in_activemodel
Prefer 'each' instead of 'for in'

@josevalim josevalim merged commit 95bd199 into rails:master May 19, 2011

@jake3030 jake3030 pushed a commit to jake3030/rails that referenced this pull request Jun 28, 2011

@roidrage @dhh roidrage + dhh Fixed plugin generator so that generated unit tests would subclass Ac…
…tiveSupport::TestCase, also introduced a helper script to reduce the needed require statements. [#1137 state:committed]

Signed-off-by: David Heinemeier Hansson <>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment