FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Added examples to tzfile docstring by weatherpattern · Pull Request #671 · dateutil/dateutil · GitHub

Added examples to tzfile docstring - #671

Merged
pganssle merged 2 commits into
dateutil:masterfrom
weatherpattern:examples-improvement-tzfile
Apr 25, 2018
Merged

pganssle merged 2 commits into
dateutil:masterfrom
weatherpattern:examples-improvement-tzfile

Conversation

Copy link
Copy Markdown
Contributor

Added examples to the tzfile to the docstring.
Note sure if there was a format you wanted.

Relates to #443

Copy link
Copy Markdown
Member

I think these need to be in .. code-block or .. doctest blocks.

That said, I don't think people are supposed to directly create tzfile objects, so I think these examples might need a bit of context. I can handle that part, though.

weatherpattern commented Apr 15, 2018
edited
Loading

Copy link
Copy Markdown
Contributor Author

Thanks for taking a look.
I am not familiar this doctest blocks but based on the sphinx doctest documentation, I tried to match the example in tzrange.

If you'd like any other changes, please let me know.

pganssle added this to the 2.7.3 milestone Apr 23, 2018
pganssle force-pushed the examples-improvement-tzfile branch 2 times, most recently from 902636d to 990d873 Compare April 24, 2018 19:05
pganssle force-pushed the examples-improvement-tzfile branch from 990d873 to 0c38900 Compare April 24, 2018 23:52
pganssle merged commit 7a99868 into dateutil:master Apr 25, 2018
pganssle mentioned this pull request May 9, 2018
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
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.

2 participants


Back | FazBrowse Home | New Git URL