Fix compatibility with AS Concern

tycooon / memery

A gem for memoization in Ruby

MIT License

177 stars 13 forks source link

Fix compatibility with AS Concern #26

Closed tycooon closed 4 years ago

tycooon commented 4 years ago

Fixes the following exception when using Memery in a ActiveSupport::Concern module:

ArgumentError:
  wrong number of arguments (given 0, expected 1)
# ./lib/memery.rb:15:in `included'

tycooon commented 4 years ago

@AlexWayfer what do you think? It's pretty common to use both AS::Concern and Memery in the same module, so I think that we should better drop that auto-include feature.

AlexWayfer commented 4 years ago

Done. You can merge #27 into this branch and then merge this PR if everything is OK.