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

[bugfix] More predictable month diffs at end-of-month #4717

Closed
wants to merge 2 commits into from
Closed

[bugfix] More predictable month diffs at end-of-month #4717

wants to merge 2 commits into from

Conversation

asenousy
Copy link
Contributor

@asenousy asenousy commented Jul 25, 2018

this resolves both #3980 & #3029 covering all end of month diff edge cases
here is codepen for bugs fixed

this is my first ever pull request :) so please let me know if I need to adjust anything, thanks

@jsf-clabot
Copy link

jsf-clabot commented Jul 25, 2018

CLA assistant check
All committers have signed the CLA.

@coveralls
Copy link

coveralls commented Jul 25, 2018

Coverage Status

Coverage increased (+0.3%) to 88.414% when pulling fac7cae on asenousy:end-of-month-diff-bugs into 2d52ae7 on moment:develop.

@marwahaha marwahaha changed the title [bugfix] end of months diff bugs [bugfix] end of months diff bugs Oct 21, 2018
@asenousy
Copy link
Contributor Author

just rebased it, to make all builds pass :)

src/lib/moment/diff.js Outdated Show resolved Hide resolved
@icambron icambron removed their assignment May 22, 2019
@toninorair
Copy link

are there any plans to merge this? Thank you

@asenousy
Copy link
Contributor Author

That would be great, @kakasahebKK @ichernev @marwahaha please let me know if there is anything more I can do to get this merged :) thanks

@ichernev
Copy link
Contributor

Merged in 813eb32

@ichernev ichernev changed the title [bugfix] end of months diff bugs [bugfix] More predictable month diffs at end-of-month Apr 29, 2020
@ichernev ichernev closed this Apr 29, 2020
ichernev added a commit that referenced this pull request Apr 29, 2020
[bugfix] More predictable month diffs at end-of-month
@ichernev
Copy link
Contributor

I mostly took the tests and used different, more understandable logic to achieve the same result.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

7 participants