Skip to content

Commit 4e21eb4

Browse files
richardlauMylesBorins
authored andcommitted
build: lint commit message in separate Travis job
Move the first commit message linting to a separate Travis job. Run the script in bash debug mode to capture any issues communicating with the GitHub API (e.g. network issues, rate limits). PR-URL: #24254 Reviewed-By: Rich Trott <[email protected]> Reviewed-By: Refael Ackermann <[email protected]> Reviewed-By: Matheus Marchini <[email protected]>
1 parent a6f786d commit 4e21eb4

File tree

1 file changed

+8
-4
lines changed

1 file changed

+8
-4
lines changed

.travis.yml

+8-4
Original file line numberDiff line numberDiff line change
@@ -4,17 +4,21 @@ cache: ccache
44
os: linux
55
matrix:
66
include:
7+
- name: "First commit message adheres to guidelines at <a href=\"https://goo.gl/p2fr5Q\">https://goo.gl/p2fr5Q</a>"
8+
if: type = pull_request
9+
language: node_js
10+
node_js: "node"
11+
script:
12+
- if [ "${TRAVIS_PULL_REQUEST}" != "false" ]; then
13+
bash -x tools/lint-pr-commit-message.sh ${TRAVIS_PULL_REQUEST};
14+
fi
715
- name: "Linter"
816
language: node_js
917
node_js: "node"
1018
env:
1119
- NODE=$(which node)
1220
script:
1321
- make lint
14-
# Lint the first commit in the PR.
15-
- if [ "${TRAVIS_PULL_REQUEST}" != "false" ]; then
16-
bash tools/lint-pr-commit-message.sh ${TRAVIS_PULL_REQUEST} || true;
17-
fi
1822
- name: "Test Suite"
1923
addons:
2024
apt:

0 commit comments

Comments
 (0)