Skip to content

minor doc improvements #2006

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

Merged
merged 14 commits into from
Apr 8, 2021
Merged

minor doc improvements #2006

merged 14 commits into from
Apr 8, 2021

Conversation

jugmac00
Copy link
Member

@jugmac00 jugmac00 commented Apr 8, 2021

@gaborbernat No need to review right now - I will go through the changes once more tonight. I am a bit in a hurry right now and the diff in the GitHub UI is actually better than the one on my local cli :-)

This was a quick reading through the docs - I will do a thorough proofreading again, maybe 2 weeks before a final release.

@jugmac00 jugmac00 requested a review from gaborbernat as a code owner April 8, 2021 15:34
@codecov
Copy link

codecov bot commented Apr 8, 2021

Codecov Report

Merging #2006 (5670a60) into rewrite (9608e23) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff            @@
##           rewrite    #2006   +/-   ##
========================================
  Coverage    99.83%   99.83%           
========================================
  Files          147      147           
  Lines         8286     8286           
  Branches       825      825           
========================================
  Hits          8272     8272           
  Misses           3        3           
  Partials        11       11           
Flag Coverage Δ
tests 99.83% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
src/tox/plugin/spec.py 100.00% <ø> (ø)
src/tox/session/cmd/quickstart.py 100.00% <ø> (ø)
src/tox/session/cmd/run/common.py 100.00% <ø> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 9608e23...5670a60. Read the comment docs.

@jugmac00
Copy link
Member Author

jugmac00 commented Apr 8, 2021

I tried to use a towncrier file as suggested in the docs a la "something.trivial" but this was not permitted by pre-commit :-D

changelog filenames......................................................Failed
- hook id: changelogs-rst
- exit code: 1

changelog files must be named ####.(feature|bugfix|doc|removal|misc).rst

docs/changelog/minor.doc.improvements.trival

@gaborbernat
Copy link
Member

changelog files must be named ####.(feature|bugfix|doc|removal|misc).rst

Update the documentation please per this rule.

@jugmac00
Copy link
Member Author

jugmac00 commented Apr 8, 2021

@gaborbernat, should be ready to review!

@gaborbernat gaborbernat merged commit 1a50180 into rewrite Apr 8, 2021
@gaborbernat gaborbernat deleted the minor-doc-improvements branch April 8, 2021 22:35
jugmac00 added a commit that referenced this pull request Apr 9, 2021
This fixes #2007

Part of the #2007 was already fixed by #2006, ie the paragraph about how
to create pseudo towncrier snippets was removed.
gaborbernat pushed a commit that referenced this pull request Apr 9, 2021
This fixes #2007

Part of the #2007 was already fixed by #2006, ie the paragraph about how
to create pseudo towncrier snippets was removed.
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