From 1b456c0b2069649696208e7c33d31f8ef5c7aee9 Mon Sep 17 00:00:00 2001 From: Casper da Costa-Luis Date: Thu, 22 Oct 2020 14:01:57 +0100 Subject: [PATCH 1/5] fix pre-commit --- .pre-commit-config.yaml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/.pre-commit-config.yaml b/.pre-commit-config.yaml index c5655a3205..32fee3e553 100644 --- a/.pre-commit-config.yaml +++ b/.pre-commit-config.yaml @@ -3,4 +3,4 @@ repos: rev: master hooks: - id: clang-format - files: \.(c|cc|cxx|cpp|h|hpp|hxx|inl|txx) + files: \.(c|cc|cxx|cpp|h|hpp|hxx|inl|txx)$ From 88cf1a6aed6b2e41f3661c75a59c70e84c960824 Mon Sep 17 00:00:00 2001 From: Casper da Costa-Luis Date: Thu, 22 Oct 2020 14:09:28 +0100 Subject: [PATCH 2/5] add CI test --- .github/workflows/test.yml | 15 +++++++++++++++ 1 file changed, 15 insertions(+) create mode 100644 .github/workflows/test.yml diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml new file mode 100644 index 0000000000..d924e78790 --- /dev/null +++ b/.github/workflows/test.yml @@ -0,0 +1,15 @@ +name: Test +on: +- push +jobs: + check: + runs-on: ubuntu-latest + name: Lint + steps: + - uses: actions/checkout@v2 + - uses: actions/setup-python@v2 + with: + python-version: 3.x + - run: sudo apt-get install -yqq clang-format + - run: pip install pre-commit + - run: pre-commit run --all-files From 4952bf474390cf841cc441fb024a31853ef4d718 Mon Sep 17 00:00:00 2001 From: Casper da Costa-Luis Date: Thu, 22 Oct 2020 14:18:41 +0100 Subject: [PATCH 3/5] use pre-commit action --- .github/workflows/test.yml | 5 +---- 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml index d924e78790..588a5120a3 100644 --- a/.github/workflows/test.yml +++ b/.github/workflows/test.yml @@ -8,8 +8,5 @@ jobs: steps: - uses: actions/checkout@v2 - uses: actions/setup-python@v2 - with: - python-version: 3.x - run: sudo apt-get install -yqq clang-format - - run: pip install pre-commit - - run: pre-commit run --all-files + - uses: pre-commit/action@v2.0.0 From d3e00ea0e71a0a2ca1ee6ef7ea7a1a0fea52c517 Mon Sep 17 00:00:00 2001 From: Casper da Costa-Luis Date: Thu, 22 Oct 2020 14:20:34 +0100 Subject: [PATCH 4/5] add pre-commit on PR --- .github/workflows/test.yml | 1 + 1 file changed, 1 insertion(+) diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml index 588a5120a3..5b426f5cc3 100644 --- a/.github/workflows/test.yml +++ b/.github/workflows/test.yml @@ -1,6 +1,7 @@ name: Test on: - push +- pull_request jobs: check: runs-on: ubuntu-latest From b950f56476937ba9368e9a3c49260057c8f96480 Mon Sep 17 00:00:00 2001 From: Casper da Costa-Luis Date: Thu, 22 Oct 2020 23:56:55 +0100 Subject: [PATCH 5/5] CI: rename Test/Lint => Check/pre-commit --- .github/workflows/{test.yml => check.yml} | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) rename .github/workflows/{test.yml => check.yml} (87%) diff --git a/.github/workflows/test.yml b/.github/workflows/check.yml similarity index 87% rename from .github/workflows/test.yml rename to .github/workflows/check.yml index 5b426f5cc3..1ffa1ebeb8 100644 --- a/.github/workflows/test.yml +++ b/.github/workflows/check.yml @@ -1,11 +1,11 @@ -name: Test +name: Check on: - push - pull_request jobs: check: runs-on: ubuntu-latest - name: Lint + name: pre-commit steps: - uses: actions/checkout@v2 - uses: actions/setup-python@v2