Ticket #4328 task closed fixed

Opened 3 years ago

Last modified 2 years ago

Deprecate twisted.python.text.docstringLStrip

Reported by: thijs Owned by:
Priority: normal Milestone:
Component: core Keywords: easy
Cc: thijs Branch: branches/deprecate-docstringLStrip-4328
Author: cyli Launchpad Bug:

Description

No other code in Twisted uses this method and it hasn't been touched in 6 years;

"""Gets rid of unsightly lefthand docstring whitespace residue.

    You'd think someone would have done this already, but apparently
    not in 1.5.2.

    BUT since we're all using Python 2.1 now, use L{inspect.getdoc}
    instead.  I{This function should go away soon.}
    """

Change History

1

Changed 3 years ago by exarkun

  • keywords easy added

Deprecate it first, of course.

2

Changed 3 years ago by thijs

  • owner changed from thijs to cyli
  • summary changed from Remove twisted.python.text.docstringLStrip to Deprecate twisted.python.text.docstringLStrip

3

Changed 3 years ago by cyli

  • branch set to branches/deprecate-docstringLStrip-4328
  • branch_author set to cyli

(In [29347]) Branching to 'deprecate-docstringLStrip-4328'

4

Changed 3 years ago by cyli

  • keywords review added
  • owner cyli deleted

5

Changed 3 years ago by cyli

(In [29354]) Deprecate twisted.python.text.docstringLStrip.

refs #4328

6

Changed 3 years ago by cyli

  • keywords review removed

7

Changed 3 years ago by cyli

(In [29355]) Ensure that flushWarnings only flushes the warnings dealing with this particular deprecation case.

refs #4328

8

Changed 3 years ago by cyli

  • keywords review added

9

Changed 3 years ago by thijs

  • owner set to cyli
  • keywords review removed

please remove the usage of L{} in the deprecation message because its not touched by epydoc afaik.

10

Changed 3 years ago by thijs

the ref to python (2.1) should also go or be put into todays context.

11

Changed 3 years ago by cyli

(In [29406]) Removed the L{} from deprecation messages, and deprecate as of 10.2 instead of 10.1

refs #4328

12

Changed 3 years ago by cyli

  • owner cyli deleted
  • keywords review added

In regards to changing the docstringLStrip docstring, I think it should be left as is - it dates the method. Also, since it's going to be removed anyway, we probably shouldn't give the impression that we are still maintaining/modifying that method by updating the docstring.

13

Changed 3 years ago by exarkun

  • keywords review removed
  • owner set to cyli

Looks good to me. Please merge, assuming the  build results look good.

14

Changed 3 years ago by cyli

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

(In [29448]) Merge deprecate-docstringLStrip-4328 : Deprecate t.p.text.docstringLStrip

Author: cyli Reviewer: thjis, exarkun Fixes: #4328

Deprecate twisted.python.text.docstringLStrip

15

Changed 2 years ago by <automation>

  • owner cyli deleted

16

Changed 2 years ago by thijs

#5036 will remove it.

Note: See TracTickets for help on using tickets.