Moved the Memoizable module to Object. Fixes #1253

spec/lucky/memoize_spec.cr

@@ -2,8 +2,9 @@ require "../spec_helper"
 
 include ContextHelper
 
+# NOTE: The Memoizable module is included in the Object charm
+# which gives all Objects access to `memoize`
 private class ObjectWithMemoizedMethods
-  include Lucky::Memoizable
   getter times_method_1_called = 0
   getter times_method_2_called = 0
   getter times_method_3_called = 0

src/charms/object.cr

@@ -1,3 +1,4 @@
 class Object
   include ::Lucky::QuickDef
+  include ::Lucky::Memoizable
 end

src/lucky.cr

@@ -4,6 +4,7 @@ require "exception_page"
 require "habitat"
 require "cry"
 require "dexter"
+require "./lucky/memoizable"
 require "./lucky/quick_def"
 require "./charms/*"
 require "http/server"

src/lucky/action.cr

@@ -11,7 +11,6 @@ abstract class Lucky::Action
   include Lucky::ActionDelegates
   include Lucky::RequestTypeHelpers
   include Lucky::Exposable
-  include Lucky::Memoizable
   include Lucky::Routable
   include Lucky::Renderable
   include Lucky::ParamHelpers

src/lucky/params.cr

@@ -1,6 +1,5 @@
 class Lucky::Params
   include Avram::Paramable
-  include Lucky::Memoizable
 
   @request : HTTP::Request
   @route_params : Hash(String, String) = {} of String => String

src/lucky/request_type_helpers.cr

@@ -2,8 +2,6 @@
 #
 # Generally the `Accept` header is checked, but some check other headers, such as `X-Requested-With`.
 module Lucky::RequestTypeHelpers
-  include Lucky::Memoizable
-
   private def default_format
     {% raise <<-TEXT
     Must set 'accepted_formats' or 'default_format' in #{@type} (or its parent class).