diff options
| author | Xavier Noria <fxn@hashref.com> | 2019-02-13 19:09:35 +0100 |
|---|---|---|
| committer | Xavier Noria <fxn@hashref.com> | 2019-02-13 19:15:04 +0100 |
| commit | b4dd69e59b3cb5c34bd58339d5b9a749e2cd82f3 (patch) | |
| tree | 00db80c5a3126dffa8ffd3762e471791b2f609f5 /activemodel/test/validators/email_validator.rb | |
| parent | 4e4b1d05ea820ff777e4734596bb6b0d85495c42 (diff) | |
| download | rails-b4dd69e59b3cb5c34bd58339d5b9a749e2cd82f3.tar.gz rails-b4dd69e59b3cb5c34bd58339d5b9a749e2cd82f3.tar.bz2 rails-b4dd69e59b3cb5c34bd58339d5b9a749e2cd82f3.zip | |
Refactors a loop
I believe the current style does not clearly communicate that we are
ignoring non-existing autoload paths altogether. Your eyes may even be
looking for an else clause that is easy to miss but that does not exist.
With the early `next` and code comment the loop reads better for my
taste.
Diffstat (limited to 'activemodel/test/validators/email_validator.rb')
0 files changed, 0 insertions, 0 deletions