Skip to content

Commit

Permalink
Merge pull request #125 from sachinkumarsingh092/optional-spelling-bo…
Browse files Browse the repository at this point in the history
…ilerplate-checks

Make the spelling and boilerplate checks optional
  • Loading branch information
k8s-ci-robot committed Jan 15, 2021
2 parents 5f06d02 + ad29307 commit 9289fd1
Show file tree
Hide file tree
Showing 3 changed files with 6 additions and 8 deletions.
6 changes: 2 additions & 4 deletions build.make
Expand Up @@ -278,15 +278,13 @@ check-go-version-%:

# Test for spelling errors.
.PHONY: test-spelling
test: test-spelling
test-spelling:
@ echo; echo "### $@:"
@ ./release-tools/verify-spelling.sh
@ ./release-tools/verify-spelling.sh "$(pwd)"

# Test the boilerplates of the files.
.PHONY: test-boilerplate
test: test-boilerplate
test-boilerplate:
@ echo; echo "### $@:"
@ ./release-tools/verify-boilerplate.sh
@ ./release-tools/verify-boilerplate.sh "$(pwd)"

4 changes: 2 additions & 2 deletions verify-boilerplate.sh
Expand Up @@ -25,8 +25,8 @@ if [[ -z "$(command -v python)" ]]; then
update-alternatives --install /usr/bin/python python /usr/bin/python3 1
fi

# The csi-release-tools directory.
TOOLS="$(dirname "${BASH_SOURCE[0]}")"
# The csi-release-tools directory (absolute path).
TOOLS="$(cd "$(dirname "${BASH_SOURCE[0]}")" && pwd -P)"

# Directory to check. Default is the parent of the tools themselves.
ROOT="${1:-${TOOLS}/..}"
Expand Down
4 changes: 2 additions & 2 deletions verify-spelling.sh
Expand Up @@ -20,8 +20,8 @@ set -o pipefail

TOOL_VERSION="v0.3.4"

# The csi-release-tools directory.
TOOLS="$(dirname "${BASH_SOURCE[0]}")"
# The csi-release-tools directory (absolute path).
TOOLS="$(cd "$(dirname "${BASH_SOURCE[0]}")" && pwd -P)"

# Directory to check. Default is the parent of the tools themselves.
ROOT="${1:-${TOOLS}/..}"
Expand Down

0 comments on commit 9289fd1

Please sign in to comment.