[Django] #27881: Add "unified diff" output mode to diffsettings management command

10 views
Skip to first unread message

Django

unread,
Feb 24, 2017, 9:26:21 PM2/24/17
to django-...@googlegroups.com
#27881: Add "unified diff" output mode to diffsettings management command
-----------------------------------------+------------------------
Reporter: Chris Lamb | Owner: nobody
Type: New feature | Status: new
Component: Uncategorized | Version: master
Severity: Normal | Keywords:
Triage Stage: Unreviewed | Has patch: 1
Needs documentation: 0 | Needs tests: 0
Patch needs improvement: 0 | Easy pickings: 0
UI/UX: 0 |
-----------------------------------------+------------------------
Most people are used to unified diffs vs. the somewhat strange "###"
formatting of diffsettings, so this adds an optional —output=unified.

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

Django

unread,
Feb 24, 2017, 9:27:26 PM2/24/17
to django-...@googlegroups.com
#27881: Add "unified diff" output mode to diffsettings management command
-------------------------------+--------------------------------------

Reporter: Chris Lamb | Owner: nobody
Type: New feature | Status: new
Component: Uncategorized | Version: master
Severity: Normal | Resolution:

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

Comment (by Chris Lamb):

[https://github.com/django/django/pull/8113 PR]

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

Django

unread,
Feb 25, 2017, 5:11:42 AM2/25/17
to django-...@googlegroups.com
#27881: Add "unified diff" output mode to diffsettings management command
-------------------------------------+-------------------------------------

Reporter: Chris Lamb | Owner: nobody
Type: New feature | Status: new
Component: Core (Management | Version: master
commands) |
Severity: Normal | Resolution:
Keywords: | Triage Stage: Accepted

Has patch: 1 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 0 | UI/UX: 0
-------------------------------------+-------------------------------------
Changes (by Claude Paroz):

* component: Uncategorized => Core (Management commands)
* stage: Unreviewed => Accepted


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

Django

unread,
Feb 25, 2017, 8:00:28 AM2/25/17
to django-...@googlegroups.com
#27881: Add "unified diff" output mode to diffsettings management command
-------------------------------------+-------------------------------------

Reporter: Chris Lamb | Owner: nobody
Type: New feature | Status: new
Component: Core (Management | Version: master
commands) |
Severity: Normal | Resolution:
Keywords: | Triage Stage: Accepted
Has patch: 1 | Needs documentation: 1

Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 0 | UI/UX: 0
-------------------------------------+-------------------------------------
Changes (by Tim Graham):

* needs_docs: 0 => 1


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

Django

unread,
Apr 7, 2017, 9:22:23 AM4/7/17
to django-...@googlegroups.com
#27881: Add "unified diff" output mode to diffsettings management command
-------------------------------------+-------------------------------------
Reporter: Chris Lamb | Owner: Haris
| Ibrahim K. V.
Type: New feature | Status: assigned

Component: Core (Management | Version: master
commands) |
Severity: Normal | Resolution:
Keywords: | Triage Stage: Accepted
Has patch: 1 | Needs documentation: 1

Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 0 | UI/UX: 0
-------------------------------------+-------------------------------------
Changes (by Haris Ibrahim K. V.):

* owner: nobody => Haris Ibrahim K. V.
* status: new => assigned


Comment:

Added documentation and here is the new PR:
https://github.com/django/django/pull/8326

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

Django

unread,
Apr 22, 2017, 10:49:45 AM4/22/17
to django-...@googlegroups.com
#27881: Add "unified diff" output mode to diffsettings management command
-------------------------------------+-------------------------------------
Reporter: Chris Lamb | Owner: (none)

Type: New feature | Status: new
Component: Core (Management | Version: master
commands) |
Severity: Normal | Resolution:
Keywords: | Triage Stage: Accepted
Has patch: 1 | Needs documentation: 1

Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 0 | UI/UX: 0
-------------------------------------+-------------------------------------
Changes (by Haris Ibrahim K. V.):

* status: assigned => new
* owner: Haris Ibrahim K. V. => (none)


Comment:

Not maintaining the PR any more. Please do feel free to fork and submit a
new PR.

--
Ticket URL: <https://code.djangoproject.com/ticket/27881#comment:5>

Django

unread,
Apr 22, 2017, 1:44:08 PM4/22/17
to django-...@googlegroups.com
#27881: Add "unified diff" output mode to diffsettings management command
-------------------------------------+-------------------------------------
Reporter: Chris Lamb | Owner: (none)
Type: New feature | Status: new
Component: Core (Management | Version: master
commands) |
Severity: Normal | Resolution:
Keywords: | Triage Stage: Accepted
Has patch: 1 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 0 | UI/UX: 0
-------------------------------------+-------------------------------------
Changes (by Tim Graham):

* needs_docs: 1 => 0


--
Ticket URL: <https://code.djangoproject.com/ticket/27881#comment:6>

Django

unread,
Apr 23, 2017, 4:23:05 AM4/23/17
to django-...@googlegroups.com
#27881: Add "unified diff" output mode to diffsettings management command
-------------------------------------+-------------------------------------
Reporter: Chris Lamb | Owner: Chris
| Lamb

Type: New feature | Status: assigned
Component: Core (Management | Version: master
commands) |
Severity: Normal | Resolution:
Keywords: | Triage Stage: Accepted
Has patch: 1 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 0 | UI/UX: 0
-------------------------------------+-------------------------------------
Changes (by Chris Lamb):

* status: new => assigned

* owner: (none) => Chris Lamb


Comment:

As the original author, am happy to take this back. I believe it's ready
for check in, however.

--
Ticket URL: <https://code.djangoproject.com/ticket/27881#comment:7>

Django

unread,
Apr 23, 2017, 11:34:16 AM4/23/17
to django-...@googlegroups.com
#27881: Add "unified diff" output mode to diffsettings management command
-------------------------------------+-------------------------------------
Reporter: Chris Lamb | Owner: Chris
| Lamb
Type: New feature | Status: assigned
Component: Core (Management | Version: master
commands) |
Severity: Normal | Resolution:
Keywords: | Triage Stage: Accepted
Has patch: 1 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 0 | UI/UX: 0
-------------------------------------+-------------------------------------

Comment (by Claude Paroz):

<offtopic>Let's take that opportunity to congratulate you for the DPL
election :-)</offtopic>

--
Ticket URL: <https://code.djangoproject.com/ticket/27881#comment:8>

Django

unread,
May 27, 2017, 9:19:45 AM5/27/17
to django-...@googlegroups.com
#27881: Add "unified diff" output mode to diffsettings management command
-------------------------------------+-------------------------------------
Reporter: Chris Lamb | Owner: Chris
| Lamb
Type: New feature | Status: closed

Component: Core (Management | Version: master
commands) |
Severity: Normal | Resolution: fixed
Keywords: | Triage Stage: Accepted

Has patch: 1 | Needs documentation: 0
Needs tests: 0 | Patch needs improvement: 0
Easy pickings: 0 | UI/UX: 0
-------------------------------------+-------------------------------------
Changes (by Tim Graham <timograham@…>):

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


Comment:

In [changeset:"2f09a285582cb475a9d27067808d9beefa7f9fba" 2f09a285]:
{{{
#!CommitTicketReference repository=""
revision="2f09a285582cb475a9d27067808d9beefa7f9fba"
Fixed #27881 -- Added diffsettings --output option.

Thanks Haris Ibrahim K. V. for writng docs.
}}}

Reply all
Reply to author
Forward
0 new messages