@dinks

why?

dinks commented on pull request blinkist/KahunaClient#1
@dinks

I strongly suggest you use options and then options[:key] etc instead of 7 parameters

@dinks
  • @dinks ccad56a
    Refactor code to be consistent
@dinks

Will do!

@dinks

+1

@dinks

What do you mean by disguise? The inheritance here of the ancestors makes sure that the method is in view and gets called.

@dinks

Well, TransactionLogger is a Module so isnt't supposed to be an extend? module A def self.included(base) p "Included #{base}" base.extend ClassMethods