Re: Patchwork v2 and beyond

On 06/24/2017 09:23 PM, Daniel Axtens wrote: > Hi all, > > So, Andrew, Russell and I had a chat about what we'd like to work > towards for future versions of Patchwork: > > * Things before v2 comes out: > ** 1 series per patch > > We think this is a solid idea. > > ** Release a tested

Re: Patchwork 2.0 ready for takeoff? (redux)

Sean Farley writes: > Daniel Axtens writes: > >> One last bug: >> >> https://lkml.org/lkml/2017/7/8/33 is being incorrectly parsed as >> https://py3.patchwork.dja.id.au/project/netdev/list/?series=4681 >> >> It looks like the square brackets are confusing the

[PATCH 4/5] tests: reuse _format_name implementation instead of code copying

Signed-off-by: Sean Farley --- patchwork/models.py| 7 --- patchwork/tests/test_series.py | 2 +- 2 files changed, 5 insertions(+), 4 deletions(-) diff --git a/patchwork/models.py b/patchwork/models.py index 4ac7a1f..e1350c2 100644 --- a/patchwork/models.py +++

[PATCH 5/5] series: fix obvious breakage for patches with ']' in the name

This copies the same regex that parse uses to find the name. Perhaps future work should abstract this into a common method. Signed-off-by: Sean Farley --- patchwork/models.py | 6 +- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/patchwork/models.py

[PATCH 2/5] models: move _format_name to module for future use

In a following patch, we'll use this method in another function. Signed-off-by: Sean Farley --- patchwork/models.py | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/patchwork/models.py b/patchwork/models.py index dcb4c55..df5c6fe 100644 ---

[PATCH 1/5] docker: pass additional args to manage.py test

Signed-off-by: Sean Farley --- tools/docker/entrypoint.sh | 6 -- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/tools/docker/entrypoint.sh b/tools/docker/entrypoint.sh index 949d8af..2f413b0 100755 --- a/tools/docker/entrypoint.sh +++

[PATCH 3/5] models: strip result of _format_name

It doesn't seem we would want names of a patch or series to start or end with a space, so let's strip that off. Signed-off-by: Sean Farley --- patchwork/models.py| 2 +- patchwork/tests/test_series.py | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff

Re: Patchwork 2.0 ready for takeoff? (redux)

Daniel Axtens writes: > One last bug: > > https://lkml.org/lkml/2017/7/8/33 is being incorrectly parsed as > https://py3.patchwork.dja.id.au/project/netdev/list/?series=4681 > > It looks like the square brackets are confusing the parser some how. What a coincidence, I have a

[PATCH] cron: fix deletion of unactivated accounts

There is a test in expire_notifications() that tries to check if the user's last login matches the date joined. (I think the login date is not set until a post-activation login.) This does not work: on patchwork.ozlabs.org there are 10k users that have never been deleted. Drop the date test: it