add second param to template handler

[emp823]

Addresses #8

DEPRECATION WARNING: Single arity template handlers are deprecated. Template handlers must
now accept two parameters, the view object and the source for the view object.
Change:
  >> Caracal::Rails::TemplateHandler.call(template)
To:
  >> Caracal::Rails::TemplateHandler.call(template, source)

[anoobbava]

@carsonbaker , Could you please update if there is any possibility of merging this PR to the master. I am having the same issue when upgrading to Rails 6.

[carsonbaker]

@anoobbava I also wish it would merge, but I'm not the maintainer.

[jdugan]

Hi, gents. Apologies for dropping the ball here. Caracal is well outside my day-to-day life now...and has been for many years, truthfully. Caracal was created for a company I worked for a decade ago and is now under the account of a different company with whom I have only a passing relationship. So, I'm the maintainer...but I'm also in kind of weird position. 🤷‍♂️

In any event, this change obviously looks fine. Because this is so small and appears to be wholly backward-compatible, I'm inclined to treat the change as a patch in terms of semantic versioning. Does that sound right to everyone else?

[emp823]

Hi, gents. Apologies for dropping the ball here. Caracal is well outside my day-to-day life now...and has been for many years, truthfully. Caracal was created for a company I worked for a decade ago and is now under the account of a different company with whom I have only a passing relationship. So, I'm the maintainer...but I'm also in kind of weird position. 🤷‍♂️

In any event, this change obviously looks fine. Because this is so small and appears to be wholly backward-compatible, I'm inclined to treat the change as a patch in terms of semantic versioning. Does that sound right to everyone else?

Works for me, thanks.

[jdugan]

caracal-rails v1.0.2 is now available from rubygems. :)

[anoobbava]

Thanks @jdugan 👍 👏