[Django] #21267: pep8 cleanup: the backslash is redundant between brackets

336 views
Skip to first unread message

Django

unread,
Oct 14, 2013, 12:24:23 PM10/14/13
to django-...@googlegroups.com
#21267: pep8 cleanup: the backslash is redundant between brackets
------------------------------------------------+------------------------
Reporter: timo | Owner: nobody
Type: Cleanup/optimization | Status: new
Component: Core (Other) | Version: master
Severity: Normal | Keywords:
Triage Stage: Accepted | Has patch: 0
Needs documentation: 0 | Needs tests: 0
Patch needs improvement: 0 | Easy pickings: 1
UI/UX: 0 |
------------------------------------------------+------------------------
To find errors: `flake8 --select=E502`

Attaching a patch to get you started.

Apply it:

`patch -p1 -i E502.diff`

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

Django

unread,
Oct 14, 2013, 1:08:01 PM10/14/13
to django-...@googlegroups.com
#21267: pep8 cleanup: the backslash is redundant between brackets
--------------------------------------+------------------------------------

Reporter: timo | Owner: nobody
Type: Cleanup/optimization | Status: new
Component: Core (Other) | Version: master
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
--------------------------------------+------------------------------------
Description changed by timo:

Old description:

> To find errors: `flake8 --select=E502`
>
> Attaching a patch to get you started.
>
> Apply it:
>
> `patch -p1 -i E502.diff`

New description:

To find errors: `flake8
--exclude=./django/utils/dictconfig.py,./django/contrib/comments/*,./django/utils/unittest.py,./tests/comment_tests/*,./django/test/_doctest.py
--select=E502`

Attaching a patch to get you started.

Apply it:

`patch -p1 -i E502.diff`

--

--
Ticket URL: <https://code.djangoproject.com/ticket/21267#comment:1>

Django

unread,
Oct 14, 2013, 2:05:12 PM10/14/13
to django-...@googlegroups.com
#21267: pep8 cleanup: the backslash is redundant between brackets
--------------------------------------+------------------------------------

Reporter: timo | Owner: nobody
Type: Cleanup/optimization | Status: new
Component: Core (Other) | Version: master
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
--------------------------------------+------------------------------------
Description changed by timo:

Old description:

> To find errors: `flake8
> --exclude=./django/utils/dictconfig.py,./django/contrib/comments/*,./django/utils/unittest.py,./tests/comment_tests/*,./django/test/_doctest.py


> --select=E502`
>
> Attaching a patch to get you started.
>
> Apply it:
>
> `patch -p1 -i E502.diff`

New description:

To find errors, ensure you have an up-to-date master with the flake8
config in setup.cfg, then remove E502 from the ignore list there. You can
then run `flake8` from the directory with setup.cfg in it to list all
errors.

Attaching an example patch to get you started.

Apply it, if you'd like:

`patch -p1 -i E502.diff`

--

--
Ticket URL: <https://code.djangoproject.com/ticket/21267#comment:2>

Django

unread,
Oct 17, 2013, 8:33:08 PM10/17/13
to django-...@googlegroups.com
#21267: pep8 cleanup: the backslash is redundant between brackets
--------------------------------------+------------------------------------
Reporter: timo | Owner: alasdair
Type: Cleanup/optimization | Status: assigned

Component: Core (Other) | Version: master
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
--------------------------------------+------------------------------------
Changes (by alasdair):

* owner: nobody => alasdair
* status: new => assigned


Comment:

Pull request https://github.com/django/django/pull/1767

--
Ticket URL: <https://code.djangoproject.com/ticket/21267#comment:3>

Django

unread,
Oct 17, 2013, 8:36:32 PM10/17/13
to django-...@googlegroups.com
#21267: pep8 cleanup: the backslash is redundant between brackets
--------------------------------------+------------------------------------
Reporter: timo | Owner: alasdair
Type: Cleanup/optimization | Status: closed

Component: Core (Other) | Version: master
Severity: Normal | Resolution: fixed

Keywords: | Triage Stage: Accepted
Has patch: 0 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 1 | UI/UX: 0
--------------------------------------+------------------------------------
Changes (by Alasdair Nicol <alasdair@…>):

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


Comment:

In [changeset:"65d1d65d52a87bba22845bcb7c3c921a8789ec19"]:
{{{
#!CommitTicketReference repository=""
revision="65d1d65d52a87bba22845bcb7c3c921a8789ec19"
Fixed #21267 -- Fixed E502 pep8 warnings
}}}

--
Ticket URL: <https://code.djangoproject.com/ticket/21267#comment:4>

Reply all
Reply to author
Forward
0 new messages