From c824c7e87a0a34e413025113d47e45c48ef2140f Mon Sep 17 00:00:00 2001 From: Joe Rickerby Date: Sun, 9 Jul 2023 15:59:27 +0100 Subject: [PATCH 1/3] Ignore docs changes on more CIs --- .cirrus.yml | 2 ++ azure-pipelines.yml | 5 +++++ 2 files changed, 7 insertions(+) diff --git a/.cirrus.yml b/.cirrus.yml index 4ba969ea2..e496f9123 100644 --- a/.cirrus.yml +++ b/.cirrus.yml @@ -1,3 +1,5 @@ +skip: changesIncludeOnly('docs/*') + run_tests: &RUN_TESTS install_cibuildwheel_script: - python -m pip install -e ".[dev]" pytest-custom-exit-code diff --git a/azure-pipelines.yml b/azure-pipelines.yml index ee0d47471..e49b7cab9 100644 --- a/azure-pipelines.yml +++ b/azure-pipelines.yml @@ -1,3 +1,8 @@ +pr: + paths: + exclude: + - docs/* + jobs: - job: linux_38 timeoutInMinutes: 120 From f5960020cd553347c9d7d93f389f1de2f00cc2c6 Mon Sep 17 00:00:00 2001 From: Joe Rickerby Date: Mon, 10 Jul 2023 11:34:10 +0100 Subject: [PATCH 2/3] Add .pre-commit-config.yaml to CI ignore rules --- .cirrus.yml | 2 +- .github/workflows/test.yml | 1 + azure-pipelines.yml | 1 + 3 files changed, 3 insertions(+), 1 deletion(-) diff --git a/.cirrus.yml b/.cirrus.yml index e496f9123..edbdb1df4 100644 --- a/.cirrus.yml +++ b/.cirrus.yml @@ -1,4 +1,4 @@ -skip: changesIncludeOnly('docs/*') +skip: changesIncludeOnly('docs/*') || changesIncludeOnly('.pre-commit-config.yaml') run_tests: &RUN_TESTS install_cibuildwheel_script: diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml index 8c61be9de..ed08a6b86 100644 --- a/.github/workflows/test.yml +++ b/.github/workflows/test.yml @@ -7,6 +7,7 @@ on: pull_request: paths-ignore: - 'docs/**' + - .pre-commit-config.yaml workflow_dispatch: # allow manual runs on branches without a PR diff --git a/azure-pipelines.yml b/azure-pipelines.yml index e49b7cab9..4f3570641 100644 --- a/azure-pipelines.yml +++ b/azure-pipelines.yml @@ -2,6 +2,7 @@ pr: paths: exclude: - docs/* + - .pre-commit-config.yaml jobs: - job: linux_38 From ddbcf561dbcfafec901b7e97551f5e9887db2c70 Mon Sep 17 00:00:00 2001 From: Joe Rickerby Date: Sat, 15 Jul 2023 09:57:54 +0100 Subject: [PATCH 3/3] Correct logic with includes-only I'm assuming it takes multiple arguments because the Starlark version of the function does https://cirrus-ci.org/guide/programming-tasks/#changes_include_only --- .cirrus.yml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/.cirrus.yml b/.cirrus.yml index edbdb1df4..e6c502f92 100644 --- a/.cirrus.yml +++ b/.cirrus.yml @@ -1,4 +1,4 @@ -skip: changesIncludeOnly('docs/*') || changesIncludeOnly('.pre-commit-config.yaml') +skip: changesIncludeOnly('docs/*', '.pre-commit-config.yaml') run_tests: &RUN_TESTS install_cibuildwheel_script: