From 80e99a4f8922186569cdd9ab763caf9ca7d5148f Mon Sep 17 00:00:00 2001 From: Christian Zentgraf Date: Wed, 29 Oct 2025 19:04:51 -0400 Subject: [PATCH] fix(ci): Check format for workflow yml files The format checker is not called when yml files in the github workflow are modified but none in presto-native-execution. Thus, skipping changes that are found to be incorrect when the format check is called on the next PR. --- .github/workflows/jdbc-connector-tests.yml | 2 +- .github/workflows/prestocpp-format-and-header-check.yml | 4 ++-- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/.github/workflows/jdbc-connector-tests.yml b/.github/workflows/jdbc-connector-tests.yml index df15822db4750..342fb01ef1c25 100644 --- a/.github/workflows/jdbc-connector-tests.yml +++ b/.github/workflows/jdbc-connector-tests.yml @@ -64,4 +64,4 @@ jobs: ./mvnw install ${MAVEN_FAST_INSTALL} -am -pl :presto-mysql,:presto-postgresql - name: Run MySQL and PostgreSQL Tests if: needs.changes.outputs.codechange == 'true' - run: ./mvnw test ${MAVEN_TEST} -P ci -pl :presto-mysql,:presto-postgresql \ No newline at end of file + run: ./mvnw test ${MAVEN_TEST} -P ci -pl :presto-mysql,:presto-postgresql diff --git a/.github/workflows/prestocpp-format-and-header-check.yml b/.github/workflows/prestocpp-format-and-header-check.yml index a7685c75c7ccc..fad4a25ab82b2 100644 --- a/.github/workflows/prestocpp-format-and-header-check.yml +++ b/.github/workflows/prestocpp-format-and-header-check.yml @@ -5,7 +5,7 @@ on: pull_request: paths: - presto-native-execution/** - - .github/workflows/prestocpp-format-and-header-check.yml + - .github/workflows/*.yml jobs: prestocpp-format-and-header-check: @@ -18,6 +18,6 @@ jobs: steps: - uses: actions/checkout@f43a0e5ff2bd294095638e18286ca9a3d1956744 # v3.6.0 with: - persist-credentials: false + persist-credentials: true - uses: actions/setup-python@3542bca2639a428e1796aaa6a2ffef0c0f575566 # v3.1.4 - uses: pre-commit/action@2c7b3805fd2a0fd8c1884dcaebf91fc102a13ecd # v3.0.1