Skip to content

Commit

Permalink
handled all remaining lint warnings (#875)
Browse files Browse the repository at this point in the history
* Add workaround for 'truthy' warnings in yaml

See adrienverge/yamllint#158
for some more discussion

* Check each yamllint line length warning and clearly markup those which cannot be easily folded into shorter lines
  • Loading branch information
hamishcoleman authored Oct 25, 2021
1 parent 6c1ed02 commit cea8bf8
Show file tree
Hide file tree
Showing 3 changed files with 8 additions and 0 deletions.
2 changes: 2 additions & 0 deletions .circleci/config.yml
Original file line number Diff line number Diff line change
Expand Up @@ -25,11 +25,13 @@ jobs:
- checkout
- run:
name: Download CMake
# yamllint disable rule:line-length
command: |
$ProgressPreference = "SilentlyContinue"
Invoke-WebRequest -URI https://github.com/Kitware/CMake/releases/download/v3.16.4/cmake-3.16.4-win64-x64.zip -OutFile $Env:HOMEPATH\cmake-3.16.4-win64-x64.zip
Expand-Archive $Env:HOMEPATH\cmake-3.16.4-win64-x64.zip -DestinationPath "$Env:ProgramFiles"
Rename-Item "$Env:ProgramFiles\cmake-3.16.4-win64-x64" -NewName CMake
# yamllint enable rule:line-length
- run: .ci\install-vcpkg.ps1 "$Env:CIRCLE_WORKING_DIRECTORY"
- run: .ci\build-project.ps1
workflows:
Expand Down
2 changes: 2 additions & 0 deletions .github/workflows/cmake-linux.yml
Original file line number Diff line number Diff line change
@@ -1,6 +1,7 @@
---
name: CMake

# yamllint disable-line rule:truthy
on: [push]

env:
Expand All @@ -12,6 +13,7 @@ jobs:
# The CMake configure and build commands are platform agnostic and should
# work equally well on Windows or Mac. You can convert this to a matrix
# build if you need cross-platform coverage.
# yamllint disable-line rule:line-length
# See: https://docs.github.com/en/free-pro-team@latest/actions/learn-github-actions/managing-complex-workflows#using-a-build-matrix
runs-on: ${{ matrix.os }}
strategy:
Expand Down
4 changes: 4 additions & 0 deletions .github/workflows/tests.yml
Original file line number Diff line number Diff line change
@@ -1,6 +1,7 @@
---
name: Testing

# yamllint disable-line rule:truthy
on:
push:
pull_request:
Expand Down Expand Up @@ -256,6 +257,8 @@ jobs:
crossbuild-essential-${{ matrix.arch }}
- name: Configure
# The HOST_TRIPLET line is not easily foldable
# yamllint disable rule:line-length
run: |
# This will warn about CC, but we cannot set CC until we run it :-S
HOST_TRIPLET=$(dpkg-architecture -a${{ matrix.arch }} -q DEB_HOST_GNU_TYPE)
Expand All @@ -265,6 +268,7 @@ jobs:
./configure --host $HOST_TRIPLET
cd packages/debian/
./configure EXTN=${{ matrix.arch }}
# yamllint enable rule:line-length

- name: Build
run: |
Expand Down

0 comments on commit cea8bf8

Please sign in to comment.