Skip to content

Commit 9b874d6

Browse files
galakAnas Nashif
authored andcommitted
CI: Fix the git commit range we pass on to gitlint
The check-compliance.py script uses gitlint to verify the commit message and takes a range of git commits to validate via the env var ${COMMIT_RANGE}. Previously we used ${SHIPPABLE_COMMIT_RANGE} however for branches that doesn't get us what we want. So we move to using origin/${PULL_REQUEST_BASE_BRANCH}..${COMMIT}. Signed-off-by: Kumar Gala <[email protected]>
1 parent a1ff1a0 commit 9b874d6

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

.shippable.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -26,8 +26,6 @@ build:
2626

2727
ci:
2828
- export CCACHE_DIR=${SHIPPABLE_BUILD_DIR}/ccache/.ccache
29-
- export COMMIT_RANGE=${SHIPPABLE_COMMIT_RANGE}
30-
- echo ${SHIPPABLE_COMMIT_RANGE}
3129
- source zephyr-env.sh
3230
- ccache -s --max-size=2000M
3331
- make host-tools
@@ -46,8 +44,10 @@ build:
4644
fi;
4745
- >
4846
if [ "$RUN_COMPLIANCE" = "1" -a "$IS_PULL_REQUEST" = "true" ]; then
47+
export COMMIT_RANGE=origin/${PULL_REQUEST_BASE_BRANCH}..${COMMIT}
4948
echo "Building a Pull Request";
5049
echo "- Building Documentation";
50+
echo "Commit range:" ${COMMIT_RANGE}
5151
make htmldocs > doc.log 2>&1;
5252
./scripts/filter-known-issues.py --config-dir .known-issues/doc/ doc.log > doc.warnings;
5353
if [ -s doc.warnings ]; then

0 commit comments

Comments
 (0)