Add Enumerable#lazy guideline to "Collections" section#938
Add Enumerable#lazy guideline to "Collections" section#938ydakuka wants to merge 1 commit intorubocop:masterfrom
Enumerable#lazy guideline to "Collections" section#938Conversation
ydakuka
force-pushed
the
add-enumerable-lazy
branch
3 times, most recently
from
806054e to
cd1789d
Compare
README.adoc
Outdated
| "when an unknown printer took a galley of type and scrambled it to make a type specimen book." | ||
| ---- | ||
|
|
||
| == Enumerable |
There was a problem hiding this comment.
Might be good to check if something else can't be moved under this section as well, as it's kind of weird to have a section with a single entry in it.
README.adoc
Outdated
|
|
||
| == Enumerable | ||
|
|
||
| === lazy [[lazy]] |
There was a problem hiding this comment.
I'd use a more descriptive heading here - something like Leverage lazy to optimize memory usage or whatever.
README.adoc
Outdated
| SIZE = 10 | ||
| MAX = 10_000_000 | ||
|
|
||
| # bad |
There was a problem hiding this comment.
You might also want to mention that this will instantiate a huge collection first, so it's clearer what the problem and value proposition is.
ydakuka
force-pushed
the
add-enumerable-lazy
branch
3 times, most recently
from
214df9c to
bcc9ef6
Compare
ydakuka
force-pushed
the
add-enumerable-lazy
branch
from
bcc9ef6 to
95ffb09
Compare
|
Done. Removed the standalone "Enumerable" section and moved the lazy tip into the "Collections" section — right after collection querying and before "Numbers". It now sits alongside other Enumerable-related tips (flat_map, reverse_each, count vs size, etc.). |
ydakuka
changed the title
Enumerable#lazyEnumerable#lazy guideline to "Collections" section
2 participants
No description provided.