Ticket #3372 task closed fixed

Opened 5 years ago

Last modified 2 years ago

deprecate --extra option to trial

Reported by: exarkun Owned by: cyli
Priority: low Milestone:
Component: trial Keywords: easy
Cc: cyli, thijs Branch: branches/remove-trial-extra-3372
Author: cyli Launchpad Bug:

Description

It serves no useful purpose any longer.

Change History

1

  Changed 5 years ago by glyph

  • priority changed from normal to low

Specifically, its purpose as described in its usage documentation is now accomplished by the --testmodule option.

This ticket raises an interesting issue though. I don't think there should ever be a 'deprecate X' ticket without a parallel 'remove X' ticket, at least until the removals are automated. I filed #3374 in an attempt to keep it on our radar.

(Also I'm lowering the priority because I really hope that we have more useful things to do than remove trivial options. Don't interpret it as a signal that it should not be done though; the option is kind of silly, and I don't know of any other use-cases for it.)

2

  Changed 3 years ago by thijs

  • owner changed from jml to cyli
  • cc cyli, thijs added

3

  Changed 2 years ago by cyli

  • branch set to branches/remove-trial-extra-3372
  • branch_author set to cyli

(In [30444]) Branching to 'remove-trial-extra-3372'

4

  Changed 2 years ago by cyli

(In [30446]) Twisted coding standard compliance

refs #3372

5

  Changed 2 years ago by cyli

  • owner cyli deleted
  • keywords easy review added

So "remove-trial-extra" may be an inaccurate branch description. Nevertheless...  build results

6

  Changed 2 years ago by thijs

  • owner set to cyli
  • keywords review removed
  • The man page needs a copyright update:
    Copyright \(co 2003-2010 Twisted Matrix Laboratories
    
  • can you also update the copyright in twisted.version.copyright? There's not really a policy that says it should be updated when working on a ticket that touches that part of the code (like a release) so you might as well do it here.

Cheers

7

  Changed 2 years ago by cyli

(In [30453]) Updated several copyrights

refs #3372

8

  Changed 2 years ago by cyli

(In [30454]) Oops, one more copyright.

refs #3372

9

follow-up: ↓ 10   Changed 2 years ago by cyli

  • keywords review added
  • owner cyli deleted

Not sure what you meant by twisted.version.copyright, so updated both twisted.python.versions 's copyright, and twisted.copyright.

10

in reply to: ↑ 9   Changed 2 years ago by thijs

Replying to cyli:

Not sure what you meant by twisted.version.copyright,

Sorry I meant twisted.copyright.copyright

so updated both twisted.python.versions 's copyright, and twisted.copyright.

even better, thanks.

11

  Changed 2 years ago by exarkun

Files that get changed should get their copyright date updated. Random extra files should not (including twisted/copyright.py which will get updated during a release).

12

  Changed 2 years ago by cyli

so... should I change both back?

13

  Changed 2 years ago by exarkun

If you want to, it would be preferable, but it teeters on the edge of not-worth-it. Mainly I want to clear up whether something similar should ever be done for another ticket (no it should not).

14

  Changed 2 years ago by exarkun

  • owner set to cyli
  • keywords review removed
  1. in ExtraTests, assertEquals is preferred over assertEqual
  2. Existing deprecations use Version("Twisted", ...) rather than Version("twisted", ...). I even feel like this made a real difference at some point, but I can't quite remember what the case was.

Otherwise looks good.

15

  Changed 2 years ago by cyli

(In [30490]) Rollback extra copyright changes

refs #3372

16

  Changed 2 years ago by cyli

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

(In [30494]) Merge remove-trial-extra-3372: deprecate --extra option to trial

Author: cyli Reviewer: thijs, exarkun Fixes: #3372

Deprecate --extra option to trial

17

  Changed 2 years ago by cyli

  • status changed from closed to reopened
  • resolution fixed deleted

(In [30502]) Reverse the merge because changing 'twisted' to 'Twisted' in Versions causes a failure in one of the new tests

reopens #3372

18

  Changed 2 years ago by cyli

(In [30503]) Changed the 'twisted' in Version to 'Twisted'

refs #3372

19

  Changed 2 years ago by cyli

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

(In [30506]) Merge remove-trial-extra-3372: deprecate --extra option to trial

Author: cyli Reviewer: thijs, exarkun Fixes: #3372

Deprecate --extra option to trial

Note: See TracTickets for help on using tickets.