pitrou commented on a change in pull request #7021:
URL: https://github.com/apache/arrow/pull/7021#discussion_r419995058
##########
File path: .github/workflows/cpp_cron.yml
##########
@@ -36,170 +36,69 @@ env:
jobs:
- debian:
- name: AMD64 Debian ${{ matrix.debian }} C++
+ docker:
+ name: ${{ matrix.title }}
runs-on: ubuntu-latest
if: ${{ !contains(github.event.pull_request.title, 'WIP') &&
github.repository == 'apache/arrow' }}
strategy:
fail-fast: false
matrix:
- debian: [10]
+ name:
+ - debian-10-cpp
+ - fedora-30-cpp
+ - ubuntu-16.04-cpp
+ - ubuntu-18.04-cpp
+ - ubuntu-18.04-cpp-cmake32
+ include:
+ - name: debian-10-cpp
+ image: debian-cpp
+ title: AMD64 Debian 10 C++
+ debian: 10
+ - name: fedora-30-cpp
+ image: fedora-cpp
+ title: AMD64 Fedora 30 C++
+ fedora: 30
+ - name: ubuntu-16.04-cpp
+ image: ubuntu-cpp
+ title: AMD64 Ubuntu 16.04 C++
+ ubuntu: 16.04
+ - name: ubuntu-18.04-cpp
+ image: ubuntu-cpp
+ title: AMD64 Ubuntu 18.04 C++
+ ubuntu: 18.04
+ - name: ubuntu-18.04-cpp-cmake32
+ image: ubuntu-cpp-cmake32
+ title: AMD64 Ubuntu 18.04 C++ CMake 3.2
+ ubuntu: 18.04
env:
- DEBIAN: ${{ matrix.debian }}
+ # the defaults here should correspond to the values in .env
Review comment:
Why are the defaults required, since each matrix entry lists the
required value explicitly?
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
