Skip to content

Commit 72d144f

Browse files
committed
Merge #364: ci, gha: Add "lint" job
f4149bd test: Make linter catch tabs in `*.qml` files (Hennadii Stepanov) ca04a8d ci, gha: Add "lint" job (Hennadii Stepanov) Pull request description: This PR ports the "lint" job to the GitHub Actions CI from now decommissioned Cirrus CI. Also a linter learned to catch tabs in the `*.qml` files. ACKs for top commit: pablomartin4btc: ACK f4149bd jarolrod: ACK f4149bd Tree-SHA512: 2e33c43c96653400108479428c17719f41c0ff9a405a6fe3d12d97f0dc11eea82d4bbc74ad316f5a51befee3b75b375d16c3e0eab4108e8e834054ba01f2cabb
2 parents 216781f + f4149bd commit 72d144f

File tree

2 files changed

+15
-1
lines changed

2 files changed

+15
-1
lines changed

.github/workflows/ci.yml

Lines changed: 14 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -23,6 +23,20 @@ env:
2323
MAKEJOBS: '-j10'
2424

2525
jobs:
26+
lint:
27+
name: 'lint'
28+
runs-on: ubuntu-latest
29+
steps:
30+
- uses: actions/checkout@v4
31+
with:
32+
fetch-depth: 0
33+
- run: |
34+
sudo ./ci/lint/04_install.sh
35+
echo "PATH=/python_build/bin:$PATH" >> "$GITHUB_ENV"
36+
echo "CIRRUS_PR=${{ github.event.number }}" >> "$GITHUB_ENV"
37+
- run: |
38+
./ci/lint/06_script.sh
39+
2640
macos-native-x86_64:
2741
name: 'macOS 13 native, x86_64, no depends, sqlite only, gui'
2842
# Use latest image, but hardcode version to avoid silent upgrades (and breaks).

test/lint/lint-whitespace.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -76,7 +76,7 @@ def get_diff(commit_range, check_only_code):
7676
exclude_args = [":(exclude)" + dir for dir in EXCLUDED_DIRS]
7777

7878
if check_only_code:
79-
what_files = ["*.cpp", "*.h", "*.md", "*.py", "*.sh"]
79+
what_files = ["*.cpp", "*.h", "*.md", "*.py", "*.sh", "*.qml"]
8080
else:
8181
what_files = ["."]
8282

0 commit comments

Comments
 (0)