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

Document methods in rrule module #1025

Merged
merged 2 commits into from Apr 24, 2020

Conversation

ffe4
Copy link
Member

@ffe4 ffe4 commented Apr 4, 2020

Summary of changes

The current documentation on RTD does not display any of the already documented methods in the rrule module.

  • Added options to automatically document members of rrule and rruleset
  • Used functools.wraps on decorator to preserve __doc__ attribute of ruleset methods.

The imports were unintentionally reorganized with isort by my editor, but I'd prefer keeping them that way.

Pull Request Checklist

ffe4 added 2 commits April 5, 2020 13:50
autodoc will inspect the __doc__ attribute of the method. Since the decorator
replaces the original method we use functools.wrap to copy __doc__.
@ffe4 ffe4 force-pushed the docs-rrule-inherited-members branch from ed6fe8f to 7a3f5c4 Compare April 5, 2020 11:54
@ffe4 ffe4 marked this pull request as ready for review April 5, 2020 12:12
@ffe4 ffe4 changed the title (WIP) Docs add inherited rrule members Docs add inherited rrule members Apr 5, 2020
from math import gcd
except ImportError:
from fractions import gcd
from functools import wraps
Copy link
Member Author

Choose a reason for hiding this comment

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

functools.wraps is new, the rest have been sorted by isort

@ffe4 ffe4 changed the title Docs add inherited rrule members Document methods in rrule module Apr 5, 2020
@pganssle pganssle closed this Apr 8, 2020
@pganssle pganssle reopened this Apr 8, 2020
@pganssle pganssle closed this Apr 8, 2020
@pganssle pganssle reopened this Apr 8, 2020
@pganssle pganssle closed this Apr 8, 2020
@pganssle pganssle reopened this Apr 8, 2020
@pganssle pganssle added this to the 2.8.2 milestone Apr 24, 2020
@pganssle pganssle merged commit a02e8c0 into dateutil:master Apr 24, 2020
@ffe4 ffe4 deleted the docs-rrule-inherited-members branch June 17, 2020 18:04
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

2 participants