Re: [Django] #36500: inconsistent 79 char limit for docstrings and comments

[Django]

#36500: inconsistent 79 char limit for docstrings and comments
------------------------------+----------------------------------------
Reporter: Mike Edmunds | Owner: Mike Edmunds
Type: Bug | Status: new
Component: Core (Other) | Version: dev
Severity: Normal | Resolution:
Keywords: flake8 | Triage Stage: Accepted
Has patch: 1 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 0 | UI/UX: 0
------------------------------+----------------------------------------
Changes (by Natalia Bidart):

* stage: Unreviewed => Accepted

Comment:

Thank you Mike!
--
Ticket URL: <https://code.djangoproject.com/ticket/36500#comment:9>
Django <https://code.djangoproject.com/>
The Web framework for perfectionists with deadlines.

[Django]

#36500: inconsistent 79 char limit for docstrings and comments

-------------------------------------+-------------------------------------

Reporter: Mike Edmunds | Owner: Mike
| Edmunds
Type: Bug | Status: new
Component: Core (Other) | Version: dev
Severity: Normal | Resolution:

Keywords: flake8 | Triage Stage: Ready for
| checkin

Has patch: 1 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 0 | UI/UX: 0

-------------------------------------+-------------------------------------
Changes (by Natalia Bidart):

* stage: Accepted => Ready for checkin

--
Ticket URL: <https://code.djangoproject.com/ticket/36500#comment:10>

[Django]

#36500: inconsistent 79 char limit for docstrings and comments
-------------------------------------+-------------------------------------
Reporter: Mike Edmunds | Owner: Mike
| Edmunds
Type: Bug | Status: new
Component: Core (Other) | Version: dev
Severity: Normal | Resolution:
Keywords: flake8 | Triage Stage: Ready for
| checkin
Has patch: 1 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 0 | UI/UX: 0
-------------------------------------+-------------------------------------

Comment (by nessita <124304+nessita@…>):

In [changeset:"69a93a88edb56ba47f624dac7a21aacc47ea474f" 69a93a8]:
{{{#!CommitTicketReference repository=""
revision="69a93a88edb56ba47f624dac7a21aacc47ea474f"
Refs #36500 -- Rewrapped long docstrings and block comments via a script.

Rewrapped long docstrings and block comments to 79 characters + newline
using script from https://github.com/medmunds/autofix-w505.
}}}
--
Ticket URL: <https://code.djangoproject.com/ticket/36500#comment:12>

[Django]

#36500: inconsistent 79 char limit for docstrings and comments
-------------------------------------+-------------------------------------
Reporter: Mike Edmunds | Owner: Mike
| Edmunds
Type: Bug | Status: new
Component: Core (Other) | Version: dev
Severity: Normal | Resolution:
Keywords: flake8 | Triage Stage: Ready for
| checkin
Has patch: 1 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 0 | UI/UX: 0
-------------------------------------+-------------------------------------
Comment (by nessita <124304+nessita@…>):

In [changeset:"78298b51629e14c0e472898b635bc819d47b7f27" 78298b5]:
{{{#!CommitTicketReference repository=""
revision="78298b51629e14c0e472898b635bc819d47b7f27"
Refs #36500 -- Corrected rewrapped long lines fixed via a script.

Manually reformatted some comments and docstrings where autofix_w505.py
changed the meaning of the formatting.
}}}
--
Ticket URL: <https://code.djangoproject.com/ticket/36500#comment:13>

[Django]

#36500: inconsistent 79 char limit for docstrings and comments
-------------------------------------+-------------------------------------
Reporter: Mike Edmunds | Owner: Mike
| Edmunds

Type: Bug | Status: closed

Component: Core (Other) | Version: dev

Severity: Normal | Resolution: fixed

Keywords: flake8 | Triage Stage: Ready for
| checkin
Has patch: 1 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 0 | UI/UX: 0
-------------------------------------+-------------------------------------

Changes (by nessita <124304+nessita@…>):

* resolution: => fixed
* status: new => closed

Comment:

In [changeset:"3ad0e759a637077b8b7064e3559cc70490311010" 3ad0e759]:
{{{#!CommitTicketReference repository=""
revision="3ad0e759a637077b8b7064e3559cc70490311010"
Fixed #36500 -- Set flake8 max-doc-length config to 79 columns.

Set flake8 max-doc-length to 79 to enforce smaller line length limit
on docstrings and comments (per coding-style docs).

Updated docs to clarify both requirements are enforced by flake8 and
to remove some leftover language from the pre-black era.
}}}
--
Ticket URL: <https://code.djangoproject.com/ticket/36500#comment:14>

[Django]

#36500: inconsistent 79 char limit for docstrings and comments
-------------------------------------+-------------------------------------
Reporter: Mike Edmunds | Owner: Mike
| Edmunds

Type: Bug | Status: new

Component: Core (Other) | Version: dev
Severity: Normal | Resolution:

Keywords: flake8 | Triage Stage: Ready for
| checkin
Has patch: 1 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 0 | UI/UX: 0
-------------------------------------+-------------------------------------

Comment (by nessita <124304+nessita@…>):

In [changeset:"55b0cc21310b76ce4018dd793ba50556eaf0af06" 55b0cc21]:
{{{#!CommitTicketReference repository=""
revision="55b0cc21310b76ce4018dd793ba50556eaf0af06"
Refs #36500 -- Shortened some long docstrings and comments.

Manually reformatted some long docstrings and comments that would be
damaged by the to-be-applied autofixer script, in cases where editorial
judgment seemed necessary for style or wording changes.
}}}
--
Ticket URL: <https://code.djangoproject.com/ticket/36500#comment:11>

[Django]

#36500: inconsistent 79 char limit for docstrings and comments
-------------------------------------+-------------------------------------
Reporter: Mike Edmunds | Owner: Mike
| Edmunds

Type: Bug | Status: closed

Component: Core (Other) | Version: dev

Severity: Normal | Resolution: fixed

Keywords: flake8 | Triage Stage: Ready for
| checkin
Has patch: 1 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 0 | UI/UX: 0
-------------------------------------+-------------------------------------

Comment (by GitHub <noreply@…>):

In [changeset:"f96c8f07e45f56f1f72125aca7f33eb3092f51b3" f96c8f07]:
{{{#!CommitTicketReference repository=""
revision="f96c8f07e45f56f1f72125aca7f33eb3092f51b3"
Refs #36500 -- Ignored formatting changes in git blame.
}}}
--
Ticket URL: <https://code.djangoproject.com/ticket/36500#comment:15>