This is an automated email from the ASF dual-hosted git repository.

raulcd pushed a commit to branch maint-16.0.0
in repository https://gitbox.apache.org/repos/asf/arrow.git

commit ef530eb5b3a08099cea5c106028bd18623bdd541
Author: Sutou Kouhei <[email protected]>
AuthorDate: Wed Apr 10 05:18:23 2024 +0900

    GH-41088: [CI][Crossbow] Fix GitHub Actions workflow syntax error (#41091)
    
    ### Rationale for this change
    
    We can't use multiple top-level `env:` in workflow. GH-40949 introduced a 
top-level `env:` by `macros.github_header()`. It broke workflows that already 
have top-level `env:`.
    
    ### What changes are included in this PR?
    
    Omit top-level `env:` key and reuse the top-level `env:` key generated by 
`macros.github_header()` in workflows.
    
    ### Are these changes tested?
    
    Yes.
    
    ### Are there any user-facing changes?
    
    No.
    * GitHub Issue: #41088
    
    Authored-by: Sutou Kouhei <[email protected]>
    Signed-off-by: Sutou Kouhei <[email protected]>
---
 dev/tasks/python-wheels/github.osx.yml | 3 +--
 dev/tasks/verify-rc/github.macos.yml   | 3 +--
 2 files changed, 2 insertions(+), 4 deletions(-)

diff --git a/dev/tasks/python-wheels/github.osx.yml 
b/dev/tasks/python-wheels/github.osx.yml
index e7b6d78981..cf99c84c60 100644
--- a/dev/tasks/python-wheels/github.osx.yml
+++ b/dev/tasks/python-wheels/github.osx.yml
@@ -17,8 +17,7 @@
 {% import 'macros.jinja' as macros with context %}
 
 {{ macros.github_header() }}
-
-env:
+# env: is generated by macros.github_header()
   ARROW_JEMALLOC: "{{ arrow_jemalloc }}"
   CC: "clang"
   CMAKE_BUILD_TYPE: release
diff --git a/dev/tasks/verify-rc/github.macos.yml 
b/dev/tasks/verify-rc/github.macos.yml
index 642cee2bb9..8963954dba 100644
--- a/dev/tasks/verify-rc/github.macos.yml
+++ b/dev/tasks/verify-rc/github.macos.yml
@@ -20,8 +20,7 @@
 {{ macros.github_header() }}
 
 {% set use_conda = use_conda|default(False) %}
-
-env:
+# env: is generated by macros.github_header()
   # Current oldest supported version according to https://endoflife.date/macos
   MACOSX_DEPLOYMENT_TARGET: "10.15"
 

Reply via email to