| #!/bin/bash | |
| # Copyright lowRISC contributors. | |
| # Licensed under the Apache License, Version 2.0, see LICENSE for details. | |
| # SPDX-License-Identifier: Apache-2.0 | |
| # A wrapper around lint_commits.py, used for CI. | |
| # | |
| # Expects a single argument, which is the pull request's target branch | |
| # (usually "master"). | |
| if [ $# != 1 ]; then | |
| echo >&2 "Usage: lint-commits.sh <tgt-branch>" | |
| exit 1 | |
| fi | |
| tgt_branch="$1" | |
| merge_base="$(git merge-base origin/$tgt_branch HEAD)" || { | |
| echo >&2 "Failed to find fork point for origin/$tgt_branch." | |
| exit 1 | |
| } | |
| echo "Checking commit messages since $merge_base" | |
| # Notes: | |
| # * Merge commits are not checked. We always use rebases instead of | |
| # merges to keep a linear history, which makes merge commits disappear | |
| # ultimately, making them only a CI artifact which should not be | |
| # checked. | |
| # * 'type=error' is used even for warnings. Only "errors" are shown in | |
| # the GitHub checks API. However, warnings don't return a non-zero | |
| # error code so don't fail the build step. | |
| util/lint_commits.py \ | |
| --no-merges \ | |
| --error-msg-prefix="##vso[task.logissue type=error]" \ | |
| --warning-msg-prefix="##vso[task.logissue type=error]" \ | |
| "$merge_base"..HEAD |