GitLab

We were off by one as the patch's body counts as one comment.
Signed-off-by: Arkadiusz Hiler <arkadiusz.hiler@intel.com>

Use the first patch timestamp if available, otherwise use series
completion date.
Signed-off-by: Arkadiusz Hiler <arkadiusz.hiler@intel.com>

Signed-off-by: Arkadiusz Hiler <arkadiusz.hiler@intel.com>

When a cover letter is sent as 0/0 it lands in series limbo due to
being in "incomplete" state.

Let's make it visible in the web UI and emit the completion events as
usual, but let's also display big notice what is going on.
Signed-off-by: Arkadiusz Hiler <arkadiusz.hiler@intel.com>

Whenever we are duplicating metadata about a revision let's clean the
rerun state.

Without this all revisions to series that had been rerun were stuck in
the "this is a rerun" state, even if updated version of the patches were
sent.

Fixes #31Signed-off-by: Arkadiusz Hiler <arkadiusz.hiler@intel.com>

Requests generated via `git request-pull` are not meant to be sent in
any standardized way, so users often copy the text to their email client
of choice. This can cause the line endings to be mangled into any
combination of '\r' and '\n'.

Let's be more liberal about what we take with regards to newlines.
Signed-off-by: Arkadiusz Hiler <arkadiusz.hiler@intel.com>

Add a button that marks the latest revision of a series as "skip
testing". Only the author of the series and maintainers of the project
can perform this action.

The `skip_testing` state is exposed through the JSON API:
/api/1.0/series/{int}/revisions/{int}/

The attached CI/testing infrastructure is responsible for checking this
flag.

The UI shows which revisions were skipped and who has triggered it.

Closes #27Signed-off-by: Arkadiusz Hiler <arkadiusz.hiler@intel.com>

Payload is the content of the last text part of the multipart message.

Mailing lists sometimes append their own footers as a new part. This
happens for emails that are already multipart or the ones with more
uncommon encodings.

In those cases we end up with an empty comment, as the footer is then
stripped out via clean_content().

We cannot use the commentbuf as it has anything diff-like stripped out
by parse_patch().

Fixes #39Signed-off-by: Arkadiusz Hiler <arkadiusz.hiler@intel.com>

This change will display the # of comments of the patch in patch list
view & series view.
Signed-off-by: Bhanuprakash Modem <bhanuprakash.modem@intel.com>

We are listing msg-ids for patches on their respective pages already.
Let's go a step further and list root msg-id(s) for given series.

root = message that started the series, whether it's the cover letter or
the first patch.

Closes #36Signed-off-by: Arkadiusz Hiler <arkadiusz.hiler@intel.com>

Old series do not have a completion date for series which breaks posting
results:

  File "patchwork/models.py" in _on_test_result
    1083.                   'revision_completed': revision.completed.isoformat()}

  Exception Type: AttributeError at /api/1.0/series/71839/revisions/1/test-results/
  Exception Value: 'NoneType' object has no attribute 'isoformat'

Let's just admit that it's unknown.
Signed-off-by: Arkadiusz Hiler <arkadiusz.hiler@intel.com>

Test results are listed on a series view. Some of them are expandable if
you click anywhere on their grey background.

This is not obvious and it's not indicated in any way.

This patch adds a triangle pointing down in the left corner to imply
this expand/collapse behavior.

The triangle starts pointing up when the section is expanded to indicate
that it is collapsible now.

Closes #35Signed-off-by: Arkadiusz Hiler <arkadiusz.hiler@intel.com>

The event is logged when patchwork receives new test result for series
revision or for a patch.
Signed-off-by: Arkadiusz Hiler <arkadiusz.hiler@intel.com>

Previously there was no way of telling when the series is complete other
than digging through the completion events, which is suboptimal.
Signed-off-by: Arkadiusz Hiler <arkadiusz.hiler@intel.com>

This solves the following error:

File "patchwork/views/patch.py" in patch
  82.                              % (patch.name, bundle.name, ex.message)]

Exception Type: AttributeError
Exception Value: 'Exception' object has no attribute 'message'
Signed-off-by: Arkadiusz Hiler <arkadiusz.hiler@intel.com>

Fixes: c626d5ea ("Add robots.txt")
Signed-off-by: Arkadiusz Hiler <arkadiusz.hiler@intel.com>

Disallow all crawling.
Signed-off-by: Arkadiusz Hiler <arkadiusz.hiler@intel.com>

When running "git pw apply --revision N SERIES_ID" it always says
that it applies the newest revision even though it does the correct
thing - it applies the requested one.

This patch fixes the logic to print the correct revision.

This fixes #22Signed-off-by: Bhanuprakash Modem <bhanuprakash.modem@intel.com>
Signed-off-by: Arkadiusz Hiler <arkadiusz.hiler@intel.com>

Let's address a common gotcha - clicking on a project name in the navbar
used to navigate us to the main page (i.e. `/`), since this patch it
will get us to the series list of the current project instead.

This fixes #30.
Signed-off-by: Manish Kumar <manish3.kumar@intel.com>
Reviewed-by: Martin Peres <martin.peres@linux.intel.com>
Signed-off-by: Arkadiusz Hiler <arkadiusz.hiler@intel.com>

The all-new path() does not take regexps.
Signed-off-by: Arkadiusz Hiler <arkadiusz.hiler@intel.com>

Wrong view, wrong way of passing view arguments.
Signed-off-by: Arkadiusz Hiler <arkadiusz.hiler@intel.com>

Signed-off-by: Arkadiusz Hiler <arkadiusz.hiler@intel.com>

Signed-off-by: Arkadiusz Hiler <arkadiusz.hiler@intel.com>

There were not used anyway.
Signed-off-by: Arkadiusz Hiler <arkadiusz.hiler@intel.com>

Signed-off-by: Arkadiusz Hiler <arkadiusz.hiler@intel.com>

Signed-off-by: Arkadiusz Hiler <arkadiusz.hiler@intel.com>

Signed-off-by: Arkadiusz Hiler <arkadiusz.hiler@intel.com>

Signed-off-by: Arkadiusz Hiler <arkadiusz.hiler@intel.com>

Signed-off-by: Arkadiusz Hiler <arkadiusz.hiler@intel.com>

Signed-off-by: Arkadiusz Hiler <arkadiusz.hiler@intel.com>

Signed-off-by: Arkadiusz Hiler <arkadiusz.hiler@intel.com>

Signed-off-by: Arkadiusz Hiler <arkadiusz.hiler@intel.com>

use_for_related_fields is gone in newer Django

We don't seem to use the manager on related fields anyway, so we are
good to remove it.
Signed-off-by: Arkadiusz Hiler <arkadiusz.hiler@intel.com>

Makes sense only with Python 2 which we do not support.
Signed-off-by: Arkadiusz Hiler <arkadiusz.hiler@intel.com>

We have already removed Python 2 support, let's remove the leftovers
too.
Signed-off-by: Arkadiusz Hiler <arkadiusz.hiler@intel.com>

Signed-off-by: Arkadiusz Hiler <arkadiusz.hiler@intel.com>

Removed with never versions of Django.
Signed-off-by: Arkadiusz Hiler <arkadiusz.hiler@intel.com>

Changes with never version.
Signed-off-by: Arkadiusz Hiler <arkadiusz.hiler@intel.com>

Deprecated since forever, gone with newer Django.
Signed-off-by: Arkadiusz Hiler <arkadiusz.hiler@intel.com>

Middleware is now defined slightly differently and we dropped _CLASSES
from the used middleware list in setting.
Signed-off-by: Arkadiusz Hiler <arkadiusz.hiler@intel.com>