Utility script that makes enforcing PEP8 within git's pre-commit hook as easy as possible.
This should be mandatory for all developers, IMHO. Change-Id: I83e8a7ab0a82df0fd08a2bef80295344b3a00a86
This commit is contained in:
parent
6589c44418
commit
16582ce70c
42
tools/enable-pre-commit-hook.sh
Executable file
42
tools/enable-pre-commit-hook.sh
Executable file
@ -0,0 +1,42 @@
|
|||||||
|
#!/bin/sh
|
||||||
|
|
||||||
|
# Copyright 2011 OpenStack LLC
|
||||||
|
#
|
||||||
|
# Licensed under the Apache License, Version 2.0 (the "License"); you may
|
||||||
|
# not use this file except in compliance with the License. You may obtain
|
||||||
|
# a copy of the License at
|
||||||
|
#
|
||||||
|
# http://www.apache.org/licenses/LICENSE-2.0
|
||||||
|
#
|
||||||
|
# Unless required by applicable law or agreed to in writing, software
|
||||||
|
# distributed under the License is distributed on an "AS IS" BASIS, WITHOUT
|
||||||
|
# WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the
|
||||||
|
# License for the specific language governing permissions and limitations
|
||||||
|
# under the License.
|
||||||
|
|
||||||
|
PRE_COMMIT_SCRIPT=.git/hooks/pre-commit
|
||||||
|
|
||||||
|
make_hook() {
|
||||||
|
echo "exec ./run_tests.sh -N -p" >> $PRE_COMMIT_SCRIPT
|
||||||
|
chmod +x $PRE_COMMIT_SCRIPT
|
||||||
|
|
||||||
|
if [ -w $PRE_COMMIT_SCRIPT -a -x $PRE_COMMIT_SCRIPT ]; then
|
||||||
|
echo "pre-commit hook was created successfully"
|
||||||
|
else
|
||||||
|
echo "unable to create pre-commit hook"
|
||||||
|
fi
|
||||||
|
}
|
||||||
|
|
||||||
|
# NOTE(jk0): Make sure we are in nova's root directory before adding the hook.
|
||||||
|
if [ ! -d ".git" ]; then
|
||||||
|
echo "unable to find .git; moving up a directory"
|
||||||
|
cd ..
|
||||||
|
if [ -d ".git" ]; then
|
||||||
|
make_hook
|
||||||
|
else
|
||||||
|
echo "still unable to find .git; hook not created"
|
||||||
|
fi
|
||||||
|
else
|
||||||
|
make_hook
|
||||||
|
fi
|
||||||
|
|
Loading…
Reference in New Issue
Block a user