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

Some rewriting and rearranging of docs, including more (and fixed) links #1132

Merged
merged 6 commits into from
Jul 15, 2020

Conversation

clarkgwillison
Copy link
Contributor

  • Addresses (but not closes) Improve the docs #972
  • Executed black -t py36 . && isort -rc . && flake8 with no errors
  • The change is fully covered by automated unit tests
  • Documented in docs/ as appropriate
  • Added an entry to the CHANGES file

This PR started with small fixes to links (small syntax errors that were causing links not to render in Sphinx) and some grammar changes, but ended up redoing much of the tutorial page.

The main commit is 3a1d475, which moves a good portion of the "parsing strings" section of the tutorial to a new page dedicated to explaining all the ways to define a Quantity() in Pint. Theory here is a slightly shorter tutorial with good links to more in-depth coverage will be more helpful to new users and old, but that's just a theory.

This PR also removes the testsuite from the developer reference, though it doesn't fix any of the other rendering issues in there.

Feedback humbly solicited.

giving the tutorial page on the docs a once-over for broken
Sphinx links

also:
- rewrote some sections for grammar
- changed (added) some section titles
- wrapped some long lines
this commit adds a new page to the docs about the various
ways of defining quantity objects in Pint, and moves some
of the content from the tutorial in there

not certain if it's a better arrangement, the theory is
that a more concise tutorial with links to more detailed
pages might get users up and running faster

very open to suggestions though
@clarkgwillison
Copy link
Contributor Author

@hgrecco this is ready for your review

@hgrecco
Copy link
Owner

hgrecco commented Jul 15, 2020

bors r+

@hgrecco
Copy link
Owner

hgrecco commented Jul 15, 2020

definetely a step in the right direction. Great1

@bors
Copy link
Contributor

bors bot commented Jul 15, 2020

Build succeeded:

@bors bors bot merged commit 03be054 into hgrecco:master Jul 15, 2020
@clarkgwillison clarkgwillison deleted the doc-fixes branch July 15, 2020 18:54
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.

2 participants