Skip to content

Issue#1167 add doc note and tests around things of last day of month actual desired behavior#1170

Merged
mariocj89 merged 5 commits into
dateutil:masterfrom
Mifrill:doc/issue#1167
Oct 12, 2021
Merged

Issue#1167 add doc note and tests around things of last day of month actual desired behavior#1170
mariocj89 merged 5 commits into
dateutil:masterfrom
Mifrill:doc/issue#1167

Conversation

@Mifrill

@Mifrill Mifrill commented Aug 31, 2021

Copy link
Copy Markdown
Contributor

Summary of changes

Added doc and tests to reflect the actual desired behavior around last day of month things.

Closes #1167

Pull Request Checklist

  • Changes have tests
  • Authors have been added to AUTHORS.md
  • News fragment added in changelog.d. See CONTRIBUTING.md for details

@Mifrill Mifrill changed the title Issue#1167 add note about last day month into relativedelta.rst Issue#1167 add doc note and tests around things of last day of month actual desired behavior Aug 31, 2021
@Mifrill

Mifrill commented Aug 31, 2021

Copy link
Copy Markdown
Contributor Author

#1168 (comment)

@Mifrill

Mifrill commented Aug 31, 2021

Copy link
Copy Markdown
Contributor Author

Hi, @pganssle could you review it, please?

@Mifrill

Mifrill commented Sep 25, 2021

Copy link
Copy Markdown
Contributor Author

Hey @pganssle any chance to have it merged?

@Mifrill

Mifrill commented Oct 9, 2021

Copy link
Copy Markdown
Contributor Author

@pganssle ping

@pganssle

pganssle commented Oct 9, 2021

Copy link
Copy Markdown
Member

Sorry, I have very little time for OSS these days and there are many other things on my to do list.

I appreciate that it sucks to have your PRs languish on under-maintained libraries, but you don't need to keep pinging me. I or someone else will review this before the next release, which won't be for a while.

Comment thread docs/relativedelta.rst Outdated
Comment thread docs/relativedelta.rst Outdated
@Mifrill
Mifrill requested a review from mariocj89 October 12, 2021 08:08
@mariocj89
mariocj89 merged commit 83ec82c into dateutil:master Oct 12, 2021
@Mifrill
Mifrill deleted the doc/issue#1167 branch October 12, 2021 10:03
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.

last day of the month is wrong??

3 participants