This project is archived and is in readonly mode.
[PATCH] Refactor requires
#4877
Found that all files on the folder were being required
separately.
I refactored that require.
-
require-refactor.diff
917 Bytes
-
Sebastian Martinez- Tag changed from rails3 to activesupport, rails3
- Title changed from Refactor requires to [PATCH] Refactor requires
-
Sebastian Martinez
- Assigned user changed from Yehuda Katz (wycats) to José Valim
-
José ValimWhy the way it's currently in Rails is an issue?
-
Sebastian Martinez
It's not wrong, just found that it was requiring all the files on the folder one by one and refactored it.
If we add another file to the integer folder no new require line would be needed this way. -
José Valim
- State changed from new to invalid
I find the current way much more readable. It could make sense though if we had several files.