On 04/19/2015 03:54 AM, Thomas De Schampheleire wrote:
On Sun, Apr 19, 2015 at 12:20 AM, Mads Kiilerich <m...@kiilerich.com> wrote:
On 04/18/2015 04:11 PM, Thomas De Schampheleire wrote:
# HG changeset patch
# User Thomas De Schampheleire <thomas.de.schamphele...@gmail.com>
# Date 1429265659 -7200
# Fri Apr 17 12:14:19 2015 +0200
# Node ID 6e760af6050e567239155a7a43b5ab02eddf877d
# Parent 32de32f6946110ecd0b8d2c79f0270025f3afaca
pullrequests: add support for custom pull request id prefix
Currently, a pull request id is referenced as #5, and the '#' symbol is
fixed.
This commit adds a configuration parameter 'pull_request_prefix',
defaulting
to '#', that can be used to customize the prefix.
For example, one could use 'PR-' to create references of the form 'PR-5'.
Yes, the # notation is weird.
But we can't make everything configurable. I think it would be better to
just use 'PR-' everywhere.
My use case is this:
we are planning to run different Kallithea instances, one in each of
our (geographically separated) sites. The reviews (pull requests) are
supposed to be traceable throughout history, so references to such
reviews are to have unique IDs across these different Kallithea
instances. Since they do not share the same database, there will be
several pull requests #4. Same for PR-4.
Therefore, we plan to use a pull request reference that contain a
reference to the site, for example: PR-<site>-4. This patch is a
solution that supports this idea in an upstreamable way, without too
much complexity.
Would you reconsider this patch after this background?
Hmm. Dunno ... I'm not fond of adding 'random' configuration knobs all
over ... but ...
It would be nice to have an extension mechanism. Perhaps not as
sophisticated as Mercurial's. Perhaps just a way to import a set of py
files that then could import other modules and monkeypatch them. Such
"extensions" could be very simple.
I would probably take a patch that moved the hardcoded # naming to a
method on the PullRequest db model, similar to how we have .url . It
would be very much like the existing .pull_request_id field so perhaps
name it .nice_id?
It is a bit weird that Kallithea pull request numbers are global.
Especially in a site that is hosting repos for multiple independent
users, it would make sense to have per repo numbering. Would that solve
your case? Will your repos in the different instances be named differently?
Another solution could be to tell the database to start the sequences
with different starting points (like 1000000) and perhaps add database
constraints to enforce it.
Thoughts?
/Mads
_______________________________________________
kallithea-general mailing list
kallithea-general@sfconservancy.org
http://lists.sfconservancy.org/mailman/listinfo/kallithea-general
