#33657: Customizable management command formatters.
-------------------------------------+-------------------------------------
     Reporter:  James Pic            |                    Owner:  nobody
         Type:                       |                   Status:  new
  Cleanup/optimization               |
    Component:  Core (Management     |                  Version:  4.0
  commands)                          |
     Severity:  Normal               |               Resolution:
     Keywords:                       |             Triage Stage:  Accepted
    Has patch:  0                    |      Needs documentation:  0
  Needs tests:  0                    |  Patch needs improvement:  0
Easy pickings:  1                    |                    UI/UX:  0
-------------------------------------+-------------------------------------

Comment (by James Pic):

 Looks good but I don't see a reason for keeping a default that swallows
 newlines because PEP257 forbids having a multiline sentence on the first
 line anyway:

 > Multi-line docstrings consist of a summary line just like a one-line
 docstring, followed by a blank line, followed by a more elaborate
 description.

 As such, the default formater which purpose is to unwrap the first
 sentence encourages breaking PEP 257.

 And users who are naturally complying with PEP257 will have to override
 the formatter, it should be the other way around.

-- 
Ticket URL: <https://code.djangoproject.com/ticket/33657#comment:3>
Django <https://code.djangoproject.com/>
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/010701805fa73284-c55ad665-80c7-4ffe-93d7-8ec6c69c63fb-000000%40eu-central-1.amazonses.com.

Reply via email to