WIP flag for merge requests

[Jim MacArthur <jim macarthur codethink co uk>]

Our current use of the 'WIP' flag - which so far as I can tell hasn't 
been documented on the mailing list - differs from the description in 
HACKING.rst.

Codethink have been using this flag to indicate that a merge request is 
ready for general review and not for maintainer review. After an initial 
review the WIP flag would be removed which signalled that it was ready 
for a more formal review. We started using this two-stage review process 
to try and reduce the load on Tristan Van Berkom.

However, HACKING.rst, says "Merge requests that are not yet ready for 
review must be prefixed with the WIP: identifier."

It's also been suggested that things not ready for review should not be 
made into a merge request.

HACKING.rst should reflect the actual procedure - what should it be?

--
--
Codethink privacy policy: https://www.codethink.co.uk/privacy.html