Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | Implement AS::Dependencies.verbose= compatibility for :zeitwerk mode | Xavier Noria | 2019-02-15 | 1 | -0/+5 |
| | |||||
* | Replace autoloader accessors with Rails.autoloaders.{main,once} | Xavier Noria | 2019-02-14 | 1 | -6/+4 |
| | | | | | | | | | | | | | | | | Rails.autoloader and Rails.once_autoloader was just tentative API good enough for a first patch. Rails.autoloader is singular and does not convey in its name that there is another autoloader. That might be confusing, for example if you set a logger and miss traces. On the other hand, the name `once_autoloader` is very close to being horrible. Rails.autoloaders.main and Rails.autoloaders.once read better for my taste, and have a nice symmetry. Also, both "main" and "once" are four letters long, short and same length. They are tagged as "rails.main" and "rails.once", respectively. References #35235. | ||||
* | Refactors a loop | Xavier Noria | 2019-02-13 | 1 | -6/+8 |
| | | | | | | | | | 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. | ||||
* | style | Xavier Noria | 2019-02-12 | 1 | -0/+2 |
| | |||||
* | Zeitwerk integration | Xavier Noria | 2019-02-12 | 1 | -0/+71 |