Skip to content

Conversation

@alfie-max
Copy link

No description provided.

gem.license = 'MIT'

gem.add_runtime_dependency('activesupport', '~> 5.1')
gem.add_runtime_dependency('activesupport')

Choose a reason for hiding this comment

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

It's better to do something like 'activesupport', '>= 5.1', '< 7', because there is no warranty that this gem will work with 7th version, or 8th, and so on.

Copy link
Author

Choose a reason for hiding this comment

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

updated

@swelther
Copy link

Any news on this? Would be very helpful @tmlee :)

@AlexWayfer
Copy link

AlexWayfer commented Oct 15, 2020

Any news on this? Would be very helpful @tmlee :)

I've switched to https://github.com/excpt/moments with some contributions (improvements) to it (already merged), so I can recommend it. It doesn't have activesupport dependency at all.

@swelther
Copy link

Any news on this? Would be very helpful @tmlee :)

I've switched to https://github.com/excpt/moments with some contributions (improvements) to it (already merged), so I can recommend it. It doesn't have activesupport dependency at all.

Thank you, that could be indeed a good alternative for us :)

@joshuapinter
Copy link
Contributor

Just a heads up, I did a quick comparison between TimeDifference and Moments to ensure similar results but I couldn't get fractional values from Moments:

irb(main):001:0> t1 = 14.days.ago
=> Thu, 04 Aug 2022 15:52:25 MDT -06:00
irb(main):002:0> t2 = Time.zone.now
=> Thu, 18 Aug 2022 15:52:31 MDT -06:00
irb(main):004:0> TimeDifference.between( t1, t2 ).in_minutes
=> 20160.09

irb(main):008:0> Moments.difference( t1, t2 ).in_minutes
=> 20160

Again, maybe there is a setting to preserve this kind of precision but I couldn't find it. Just be aware. I think I'll try a fork of this library as a quick fix for now.

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.

4 participants