[ovs-dev] [PATCH] committer-responsibilities: Add guidelines for fixing up mistakes.
i.maximets at samsung.com
Wed Jul 24 17:25:54 UTC 2019
On 24.07.2019 20:05, Ben Pfaff wrote:
> Signed-off-by: Ben Pfaff <blp at ovn.org>
> Based on an off-lst discussion with Ilya. This is just my first, off-hand
> thought on the topic. Feedback welcome!
We could also consider suggesting a git hook like that:
$ cat .git/hooks/pre-push
if [ $protected_remote != $remote ]; then exit 0; fi
echo "You're about to push to $protected_remote."
read -p "Do you want to proceed? [y|n] " reply < /dev/tty
if echo $reply | grep -E '^[Yy]$' > /dev/null; then exit 0; fi
> diff --git a/Documentation/internals/committer-responsibilities.rst b/Documentation/internals/committer-responsibilities.rst
> index 4d10c3980875..b34dddaa99fd 100644
> --- a/Documentation/internals/committer-responsibilities.rst
> +++ b/Documentation/internals/committer-responsibilities.rst
> @@ -94,3 +94,9 @@ Use Reported-by: and Tested-by: tags in commit messages to indicate the
> source of a bug report.
> Keep the ``AUTHORS.rst`` file up to date.
> +If you mistakenly push a commit that should not have been applied, it's OK to
> +use a Git "force-push" to remove it from the tree, as long as you do it within
> +a few minutes and no one has pushed other commits to the tree in the meantime.
> +Otherwise, it's necessary to revert the commit. (If you do this frequently,
> +more than once or twice a year, then maybe you should be more careful.)
More information about the dev