-
Notifications
You must be signed in to change notification settings - Fork 130
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[RFC] hooks: propose project configured hooks #908
base: master
Are you sure you want to change the base?
Conversation
There is an issue in commit 4102454: |
/submit |
Submitted as pull.908.git.1616105016055.gitgitgadget@gmail.com To fetch this version into
To fetch this version to local tag
|
On the Git mailing list, Junio C Hamano wrote (reply to this):
|
On the Git mailing list, Albert Cui wrote (reply to this):
|
User |
On the Git mailing list, "brian m. carlson" wrote (reply to this):
|
User |
On the Git mailing list, Ævar Arnfjörð Bjarmason wrote (reply to this):
|
User |
/submit |
Submitted as pull.908.v2.git.1616723016659.gitgitgadget@gmail.com To fetch this version into
To fetch this version to local tag
|
On the Git mailing list, Emily Shaffer wrote (reply to this):
|
User |
On the Git mailing list, Derrick Stolee wrote (reply to this):
|
User |
On the Git mailing list, Albert Cui wrote (reply to this):
|
On the Git mailing list, Ævar Arnfjörð Bjarmason wrote (reply to this):
|
On the Git mailing list, Ævar Arnfjörð Bjarmason wrote (reply to this):
|
On the Git mailing list, Albert Cui wrote (reply to this):
|
On the Git mailing list, Albert Cui wrote (reply to this):
|
On the Git mailing list, Junio C Hamano wrote (reply to this):
|
On the Git mailing list, Albert Cui wrote (reply to this):
|
On the Git mailing list, Albert Cui wrote (reply to this):
|
On the Git mailing list, "brian m. carlson" wrote (reply to this):
|
On the Git mailing list, Ævar Arnfjörð Bjarmason wrote (reply to this):
|
On the Git mailing list, Derrick Stolee wrote (reply to this):
|
On the Git mailing list, Albert Cui wrote (reply to this):
|
On the Git mailing list, Junio C Hamano wrote (reply to this):
|
On the Git mailing list, Ævar Arnfjörð Bjarmason wrote (reply to this):
|
On the Git mailing list, Ævar Arnfjörð Bjarmason wrote (reply to this):
|
Replying to a few different messages here via gitgitgadget, apologies for the lack of individual attributions on the quotes.
In particular, at least FreeBSD is included in Git's CI. Is there an explicit list of supported platforms (and perhaps a notion of support tiers)?
I came across this proposal while looking for this sort of thing for the FreeBSD repositories so I'll explain our use case. (We started migrating from Subversion to Git some time ago and finished the last repo a couple of weeks ago.) We use one hook today, and would like developers to have an up-to-date version: prepare-commit-msg. We have some standard commit message trailers like
I think providing messages that hooks are available or updated from clone or checkout would suit FreeBSD. |
Please reply on the list. |
On the Git mailing list, Ed Maste wrote (reply to this):
|
User |
On the Git mailing list, Junio C Hamano wrote (reply to this):
|
On the Git mailing list, Ed Maste wrote (reply to this):
|
On the Git mailing list, Junio C Hamano wrote (reply to this):
|
On the Git mailing list, "brian m. carlson" wrote (reply to this):
|
7daca71
to
e8fbcc0
Compare
/submit |
Submitted as pull.908.v3.git.1619228290023.gitgitgadget@gmail.com To fetch this version into
To fetch this version to local tag
|
On the Git mailing list, Junio C Hamano wrote (reply to this):
|
Hooks today are configured at the repository level, making it difficult to share hooks across repositories. Configuration-based hook management, by moving hooks configuration to the config, makes this much easier. However, there is still no good way for project maintainers to encourage or enforce adoption of specific hook commands on specific hook events in a repository. As such, there are many tools that provide this functionality on top of Git. This patch documents the requirements we propose for this feature as well as a design sketch for implementation. Signed-off-by: Albert Cui <albertqcui@gmail.com> Helped-by: Emily Shaffer <emilyshaffer@google.com> Helped-by: Jonathan Nieder <jrnieder@gmail.com>
/submit |
Submitted as pull.908.v4.git.1620241892929.gitgitgadget@gmail.com To fetch this version into
To fetch this version to local tag
|
On the Git mailing list, Jonathan Tan wrote (reply to this):
|
User |
On the Git mailing list, Albert Cui wrote (reply to this):
|
On the Git mailing list, Jonathan Tan wrote (reply to this):
|
On the Git mailing list, Jonathan Tan wrote (reply to this):
|
On the Git mailing list, Ævar Arnfjörð Bjarmason wrote (reply to this):
|
V4:
Signed-off-by: Albert Cui albertqcui@gmail.com
Helped-by: Emily Shaffer emilyshaffer@google.com
cc: Albert Cui albertqcui@gmail.com
cc: "brian m. carlson" sandals@crustytoothpaste.net
cc: Ævar Arnfjörð Bjarmason avarab@gmail.com
cc: Emily Shaffer emilyshaffer@google.com
cc: Derrick Stolee stolee@gmail.com
cc: Ed Maste emaste@freebsd.org
cc: Jonathan Tan jonathantanmy@google.com