[PATCH 0/5] Add Django 1.10 support

2016-10-09 Thread Stephen Finucane
Now that the parser tools have been moved to management commands, we can take the required steps to support Django 1.10. This means we now support Django 1.6 - 1.10. Stephen Finucane (5): requirements: Bump django-rest-framework to 1.4.x templates: Resolve cycle issues with Django 1.10

[PATCH 2/5] templates: Resolve cycle issues with Django 1.10

2016-10-09 Thread Stephen Finucane
The 'future.cycle' template tag is removed in Django 1.10. Since Django 1.7 is still supported by Patchwork, it is necessary to provide a custom 'cycle' wrapper to prevent the deprecation warnings resolved by '9cab078' being reintroduced. Signed-off-by: Stephen Finucane ---

[PATCH 3/5] views: Resolve select_related issues

2016-10-09 Thread Stephen Finucane
Django 1.10 prohibits non-relational fields for nested relations [1]. Replace said call with a functional alternative. [1] https://docs.djangoproject.com/en/dev/releases/1.10/#select-related-prohibits-non-relational-fields-for-nested-relations Signed-off-by: Stephen Finucane

[PATCH 1/5] requirements: Bump django-rest-framework to 1.4.x

2016-10-09 Thread Stephen Finucane
This version is compatible with all versions of Django that Patchwork is recommended for. Signed-off-by: Stephen Finucane --- patchwork/settings/dev.py | 2 +- requirements-test.txt | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git

[PATCH 5/5] travis: Enable *-django110 targets

2016-10-09 Thread Stephen Finucane
Signed-off-by: Stephen Finucane --- .travis.yml | 3 +++ 1 file changed, 3 insertions(+) diff --git a/.travis.yml b/.travis.yml index 9edd3e1..6e5fe52 100644 --- a/.travis.yml +++ b/.travis.yml @@ -14,12 +14,15 @@ env: - TOX_ENV=py27-django17 - TOX_ENV=py27-django18

Re: [PATCH v5 0/7] Add series support

2016-10-09 Thread Andrew Donnellan
On 10/10/16 09:25, Stephen Finucane wrote: Add support for series. Series are groups of patches sent as one bundle. For example: [PATCH 0/3] A cover letter [PATCH 1/3] The first patch [PATCH 2/3] The second patch [PATCH 3/3] The third patch Looks like patch 4 didn't make

Re: [PATCH 0/5] Add Django 1.10 support

2016-10-09 Thread Daniel Axtens
I have a similar series I was going to post - it looks basically identical to yours. So, this series is: Reviewed-by: Daniel Axtens Regards, Daniel Stephen Finucane writes: > I thought I'd already sent this, but clearly not. > > Stephen Finucane (5): >

Re: [PATCH] models: Remove 'UserProfile.primary_project'

2016-10-09 Thread Andrew Donnellan
On 10/10/16 09:36, Stephen Finucane wrote: This field was unused and has been for a long time (pre-0.9.0). Signed-off-by: Stephen Finucane LGTM! Reviewed-by: Andrew Donnellan -- Andrew Donnellan OzLabs, ADL Canberra

Fwd: [PATCH 0/5] Add Django 1.10 support

2016-10-09 Thread mike cho
unsubscribe -- 전달된 메일 -- 보낸사람: "Stephen Finucane" 날짜: 2016. 10. 9. 오후 3:27 제목: [PATCH 0/5] Add Django 1.10 support 받는사람: 참조: I thought I'd already sent this, but clearly not. Stephen Finucane (5): requirements: Bump

[PATCH] models: Remove 'UserProfile.primary_project'

2016-10-09 Thread Stephen Finucane
This field was unused and has been for a long time (pre-0.9.0). Signed-off-by: Stephen Finucane --- patchwork/forms.py | 2 +- .../0014_remove_userprofile_primary_project.py | 18 ++ patchwork/models.py

[PATCH v5 1/7] models: Convert functions to properties

2016-10-09 Thread Stephen Finucane
A number of models contain functions that are, semantically speaking, actually properties. Mark them as such. Signed-off-by: Stephen Finucane Reviewed-by: Andy Doan --- patchwork/models.py | 15 ++- patchwork/views/__init__.py | 4

[PATCH 5/5] travis: Enable *-django110 targets

2016-10-09 Thread Stephen Finucane
Signed-off-by: Stephen Finucane --- .travis.yml | 3 +++ 1 file changed, 3 insertions(+) diff --git a/.travis.yml b/.travis.yml index 9edd3e1..6e5fe52 100644 --- a/.travis.yml +++ b/.travis.yml @@ -14,12 +14,15 @@ env: - TOX_ENV=py27-django17 - TOX_ENV=py27-django18

[PATCH 4/5] requirements: Add support for Django 1.10

2016-10-09 Thread Stephen Finucane
All issues around 1.10 support have been resolved. Add tox targets and extend the range of supported versions to include this release. Signed-off-by: Stephen Finucane --- requirements-dev.txt | 2 +- requirements-prod.txt | 2 +- tox.ini | 3 ++- 3 files

[PATCH 3/5] views: Resolve select_related issues

2016-10-09 Thread Stephen Finucane
Django 1.10 prohibits non-relational fields for nested relations [1]. Replace said call with a functional alternative. [1] https://docs.djangoproject.com/en/dev/releases/1.10/#select-related-prohibits-non-relational-fields-for-nested-relations Signed-off-by: Stephen Finucane

[PATCH 0/5] Add Django 1.10 support

2016-10-09 Thread Stephen Finucane
I thought I'd already sent this, but clearly not. Stephen Finucane (5): requirements: Bump django-rest-framework to 1.4.x templates: Resolve cycle issues with Django 1.10 views: Resolve select_related issues requirements: Add support for Django 1.10 travis: Enable *-django110 targets

[PATCH v5 0/7] Add series support

2016-10-09 Thread Stephen Finucane
Add support for series. Series are groups of patches sent as one bundle. For example: [PATCH 0/3] A cover letter [PATCH 1/3] The first patch [PATCH 2/3] The second patch [PATCH 3/3] The third patch The following features are currently provided: Parsing === * Creation

[PATCH v5 6/7] templates: Integrate series support

2016-10-09 Thread Stephen Finucane
Integrate support for series in the web UI. This is rather straightforward, the only significant change being the addition of a filter for series filtering. Signed-off-by: Stephen Finucane --- v5: - Don't use the 'Patch.series' property, which results in a new query each

[PATCH v5 5/7] templates: Generate and use a "handle"

2016-10-09 Thread Stephen Finucane
In the absence of a name, the email is currently used. Some emails are long and rather unreadable. The likelihood of someone's localpart (i.e. the bit before the '@') conflicting with that of someone else on the same patchwork instance and project is very low, so form a "handle" from just this.

[PATCH v5 2/7] models: Add 'Series' model and related models

2016-10-09 Thread Stephen Finucane
Add a series model. This model is intentionally very minimal to allow as much dynaminism as possible. It is expected that patches will be migrated between series as new data is provided. Signed-off-by: Stephen Finucane --- v5: - Store cover letter name in SeriesRevision.name

[PATCH v5 3/7] parser: Add series parsing

2016-10-09 Thread Stephen Finucane
It is now possible to parse and store series, so do just that. The parsing at the moment is based on both RFC822 headers and subject lines. Signed-off-by: Stephen Finucane --- v4: - Update per new 'SeriesRevision'-'Patch' relationship v3: - Rework how nested series are handled