Should Lucky::FlashStore be an enumerable?

[matthewmcgarvey]

Right now we delegate a few enumerable methods to Lucky::FlashStore#all which is not accessible but there are other methods, like Enumerable#each_with_index, that users are not able to use. I'm wondering if we could keep Lucky::FlashStore#all encapsulated but include Enumerable to provide a more robust interface to it. Would that break the encapsulation or no?

lucky/src/lucky/cookies/flash_store.cr

Line 5 in 0635f76

delegate any?, each, empty?, to: all