The Wayback Machine - http://web.archive.org/web/20200910212155/https://github.com/sdispater/pendulum/pull/477
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

Add next_anniversary functions to Date and DateTime #477

Open
wants to merge 3 commits into
base: master
from

Conversation

@StephenBrown2
Copy link

StephenBrown2 commented May 29, 2020

Pull Request Check List

This adds next_anniversary to both Date and DateTime instances, returning the next anniversary for a given instance, properly accounting for leap years by taking advantage of the .add function rather than simply comparing the month and day of an instance.

  • Added tests for changed code.
  • Updated documentation for changed code.
@StephenBrown2
Copy link
Author

StephenBrown2 commented May 29, 2020

MacOS having a bad day, apparently... :-/

@StephenBrown2 StephenBrown2 force-pushed the StephenBrown2:add_next_anniversary branch from d37c667 to f453dc7 Jun 1, 2020
sdispater and others added 3 commits Jul 14, 2020
@StephenBrown2 StephenBrown2 force-pushed the StephenBrown2:add_next_anniversary branch from f453dc7 to 01053af Aug 11, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants
You can’t perform that action at this time.