Minor cleanup of retry class

isaacseymour / activejob-retry

Automatic retries for ActiveJob

MIT License

138 stars 14 forks source link

Closed greysteil closed 9 years ago

greysteil commented 9 years ago

Instance variables default to nil, so there's no need to check whether we're assigning them to nil
Requiring ActiveJob means we have access to ActiveSupport. We may as well use it!

isaacseymour commented 9 years ago

Ha, I kept getting annoyed at the lack of try yesterday...