Ticket #2309 defect closed fixed

Opened 6 years ago

Last modified 6 years ago

trial man page incorrectly explains --coverage behaviour

Reported by: jml Owned by:
Priority: highest Milestone:
Component: trial Keywords:
Cc: Branch:
Author: Launchpad Bug:

Description

Currently, it says:

 Generate  coverage  information in the given directory (relative
 to _trial_temp). Requires Python 2.3 or higher.

At the moment, trial doesn't let you set the coverage directory. It is always _trial_temp/coverage/. The man page should also have a stab at explaining what to do with the coverage output.

Change History

1

Changed 6 years ago by glyph

Trial should also let you set the coverage directory, in addition to the immediate bugfix for the docs. Is there a ticket for that? It should be linked here.

2

Changed 6 years ago by jml

No there isn't. Please file one.

3

Changed 6 years ago by exarkun

Um. Don't file that ticket, please. Trial used to let you set the coverage directory, that's why the documentation says it does. It doesn't let you do that anymore and I can't think of any reason for it to. If you can, please write up an extensive justification so that 6 months from now someone doesn't decide it would be a good idea to delete it again.

4

Changed 6 years ago by exarkun

  • owner jml deleted
  • priority changed from high to highest
  • keywords review added

Doc fix is ready for review in coverage-docs-2309

5

Changed 6 years ago by jerub

  • keywords review removed

Excellent. Please merge this documentation change.

6

Changed 6 years ago by exarkun

  • status changed from new to closed
  • resolution set to fixed

(In [19065]) Merge coverage-docs-2309

Author: exarkun Reviewer: jerub Fixes #2309

Correct docstring and man page documentation of trial's --coverage option so that it does not incorrectly suggest that a parameter is accepted by the option which specifies the coverage output directory.

7

Changed 2 years ago by <automation>

Note: See TracTickets for help on using tickets.