Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove deprecation warning in spec test #8

Merged
merged 1 commit into from
Dec 22, 2014

Conversation

dtaniwaki
Copy link
Owner

Remove the warning DEPRECATION WARNING:#deliver!is deprecated and will be removed in Rails 5. Use#deliver_now!to deliver immediately or#deliver_later!to deliver through Active Job. (called from block (3 levels) in <top (required)> at /Users/dtaniwaki/github/mandriller/spec/support/action_mailer.rb:4) in the spec test.

@@ -1,7 +1,8 @@
deliver_method = ActionMailer.version.to_s.to_f >= 4.2 ? :deliver_now! : :deliver!
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Line is too long. [82/80]

@coveralls
Copy link

Coverage Status

Coverage remained the same when pulling 64ee287 on remove-deprecation-warning into 5133208 on master.

@@ -1,7 +1,8 @@
deliver_method = ActionMailer.respond_to?(:version) && ActionMailer.version.to_s.to_f >= 4.2 ? :deliver_now! : :deliver!
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Line is too long. [120/80]

@coveralls
Copy link

Coverage Status

Coverage remained the same when pulling d65ac9e on remove-deprecation-warning into 807d181 on master.

dtaniwaki added a commit that referenced this pull request Dec 22, 2014
Remove deprecation warning in spec test
@dtaniwaki dtaniwaki merged commit d98c61c into master Dec 22, 2014
@dtaniwaki dtaniwaki deleted the remove-deprecation-warning branch December 22, 2014 02:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants