Skip to content

Commit 951a9f3

Browse files
committed
turn off submodule updating
1 parent a939918 commit 951a9f3

6 files changed

Lines changed: 110 additions & 84 deletions

File tree

Lines changed: 15 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,15 @@
1+
name: "Setup Submodules"
2+
description: "Initialize and update git submodules for testing"
3+
runs:
4+
using: "composite"
5+
steps:
6+
- name: Initialize and update submodules
7+
shell: bash
8+
run: |
9+
git config --global --add safe.directory $GITHUB_WORKSPACE
10+
git submodule init
11+
# Override update=none setting for CI
12+
git config submodule.testing.update checkout
13+
git config submodule.parquet-testing.update checkout
14+
git config submodule.datafusion-testing.update checkout
15+
git submodule update --depth 1

.github/workflows/dependencies.yml

Lines changed: 6 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,7 @@ concurrency:
2424
on:
2525
push:
2626
branches-ignore:
27-
- 'gh-readonly-queue/**'
27+
- "gh-readonly-queue/**"
2828
paths:
2929
- "**/Cargo.toml"
3030
- "**/Cargo.lock"
@@ -44,10 +44,11 @@ jobs:
4444
container:
4545
image: amd64/rust
4646
steps:
47-
- uses: actions/checkout@8e8c483db84b4bee98b60c0593521ed34d9990e8 # v6.0.1
47+
- uses: actions/checkout@8e8c483db84b4bee98b60c0593521ed34d9990e8 # v6.0.1
4848
with:
49-
submodules: true
5049
fetch-depth: 1
50+
- name: Setup Submodules
51+
uses: ./.github/actions/setup-submodules
5152
- name: Setup Rust toolchain
5253
uses: ./.github/actions/setup-builder
5354
with:
@@ -62,8 +63,8 @@ jobs:
6263
container:
6364
image: amd64/rust
6465
steps:
65-
- uses: actions/checkout@8e8c483db84b4bee98b60c0593521ed34d9990e8 # v6.0.1
66+
- uses: actions/checkout@8e8c483db84b4bee98b60c0593521ed34d9990e8 # v6.0.1
6667
- name: Install cargo-machete
6768
run: cargo install cargo-machete --version ^0.9 --locked
6869
- name: Detect unused dependencies
69-
run: cargo machete --with-metadata
70+
run: cargo machete --with-metadata

.github/workflows/docs_pr.yaml

Lines changed: 4 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -34,18 +34,18 @@ on:
3434
workflow_dispatch:
3535

3636
jobs:
37-
3837
# Test doc build
3938
linux-test-doc-build:
4039
name: Test doc build
4140
runs-on: ubuntu-latest
4241
steps:
43-
- uses: actions/checkout@8e8c483db84b4bee98b60c0593521ed34d9990e8 # v6.0.1
42+
- uses: actions/checkout@8e8c483db84b4bee98b60c0593521ed34d9990e8 # v6.0.1
4443
with:
45-
submodules: true
4644
fetch-depth: 1
45+
- name: Setup Submodules
46+
uses: ./.github/actions/setup-submodules
4747
- name: Setup Python
48-
uses: actions/setup-python@83679a892e2d95755f2dac6acb0bfd1e9ac5d548 # v6.1.0
48+
uses: actions/setup-python@83679a892e2d95755f2dac6acb0bfd1e9ac5d548 # v6.1.0
4949
with:
5050
python-version: "3.12"
5151
- name: Install doc dependencies
@@ -60,4 +60,3 @@ jobs:
6060
source venv/bin/activate
6161
cd docs
6262
./build.sh # fails on errors
63-

.github/workflows/extended.yml

Lines changed: 22 additions & 23 deletions
Original file line numberDiff line numberDiff line change
@@ -35,45 +35,45 @@ on:
3535
# support extended test suite for release candidate branches,
3636
# it is not expected to have many changes in these branches,
3737
# so running extended tests is not a burden
38-
- 'branch-*'
38+
- "branch-*"
3939
# Also run for changes to some critical areas that are most likely
4040
# to trigger errors in extended tests
4141
pull_request:
42-
branches: [ '**' ]
42+
branches: ["**"]
4343
paths:
44-
- 'datafusion/physical*/**/*.rs'
45-
- 'datafusion/expr*/**/*.rs'
46-
- 'datafusion/optimizer/**/*.rs'
47-
- 'datafusion-testing'
44+
- "datafusion/physical*/**/*.rs"
45+
- "datafusion/expr*/**/*.rs"
46+
- "datafusion/optimizer/**/*.rs"
47+
- "datafusion-testing"
4848
workflow_dispatch:
4949
inputs:
5050
pr_number:
51-
description: 'Pull request number'
51+
description: "Pull request number"
5252
type: string
5353
check_run_id:
54-
description: 'Check run ID for status updates'
54+
description: "Check run ID for status updates"
5555
type: string
5656
pr_head_sha:
57-
description: 'PR head SHA'
57+
description: "PR head SHA"
5858
type: string
5959

6060
permissions:
6161
contents: read
6262
checks: write
6363

6464
jobs:
65-
6665
# Check crate compiles and base cargo check passes
6766
linux-build-lib:
6867
name: linux build test
6968
runs-on: ubuntu-latest
7069
# note: do not use amd/rust container to preserve disk space
7170
steps:
72-
- uses: actions/checkout@8e8c483db84b4bee98b60c0593521ed34d9990e8 # v6.0.1
71+
- uses: actions/checkout@8e8c483db84b4bee98b60c0593521ed34d9990e8 # v6.0.1
7372
with:
7473
ref: ${{ github.event.inputs.pr_head_sha }} # will be empty if triggered by push
75-
submodules: true
7674
fetch-depth: 1
75+
- name: Setup Submodules
76+
uses: ./.github/actions/setup-submodules
7777
- name: Install Rust
7878
run: |
7979
curl --proto '=https' --tlsv1.2 -sSf https://sh.rustup.rs | sh -s -- -y
@@ -93,13 +93,14 @@ jobs:
9393
runs-on: ubuntu-latest
9494
# note: do not use amd/rust container to preserve disk space
9595
steps:
96-
- uses: actions/checkout@8e8c483db84b4bee98b60c0593521ed34d9990e8 # v6.0.1
96+
- uses: actions/checkout@8e8c483db84b4bee98b60c0593521ed34d9990e8 # v6.0.1
9797
with:
9898
ref: ${{ github.event.inputs.pr_head_sha }} # will be empty if triggered by push
99-
submodules: true
10099
fetch-depth: 1
100+
- name: Setup Submodules
101+
uses: ./.github/actions/setup-submodules
101102
- name: Free Disk Space (Ubuntu)
102-
uses: jlumbroso/free-disk-space@54081f138730dfa15788a46383842cd2f914a1be # v1.3.1
103+
uses: jlumbroso/free-disk-space@54081f138730dfa15788a46383842cd2f914a1be # v1.3.1
103104
- name: Install Rust
104105
run: |
105106
curl --proto '=https' --tlsv1.2 -sSf https://sh.rustup.rs | sh -s -- -y
@@ -137,11 +138,12 @@ jobs:
137138
container:
138139
image: amd64/rust
139140
steps:
140-
- uses: actions/checkout@8e8c483db84b4bee98b60c0593521ed34d9990e8 # v6.0.1
141+
- uses: actions/checkout@8e8c483db84b4bee98b60c0593521ed34d9990e8 # v6.0.1
141142
with:
142143
ref: ${{ github.event.inputs.pr_head_sha }} # will be empty if triggered by push
143-
submodules: true
144144
fetch-depth: 1
145+
- name: Setup Submodules
146+
uses: ./.github/actions/setup-submodules
145147
- name: Setup Rust toolchain
146148
uses: ./.github/actions/setup-builder
147149
with:
@@ -158,11 +160,12 @@ jobs:
158160
container:
159161
image: amd64/rust
160162
steps:
161-
- uses: actions/checkout@8e8c483db84b4bee98b60c0593521ed34d9990e8 # v6.0.1
163+
- uses: actions/checkout@8e8c483db84b4bee98b60c0593521ed34d9990e8 # v6.0.1
162164
with:
163165
ref: ${{ github.event.inputs.pr_head_sha }} # will be empty if triggered by push
164-
submodules: true
165166
fetch-depth: 1
167+
- name: Setup Submodules
168+
uses: ./.github/actions/setup-submodules
166169
- name: Setup Rust toolchain
167170
uses: ./.github/actions/setup-builder
168171
with:
@@ -171,7 +174,3 @@ jobs:
171174
run: |
172175
cargo test --features backtrace,parquet_encryption --profile release-nonlto --test sqllogictests -- --include-sqlite
173176
cargo clean
174-
175-
176-
177-

0 commit comments

Comments
 (0)