Skip to content
This repository was archived by the owner on Sep 2, 2025. It is now read-only.

Commit f25110c

Browse files
authored
[Backport 1.6.latest] Combine build verification jobs to avoid needing to upload and download artifacts (#1461) (#1467)
* combine build verification jobs to avoid needing to upload and download artifacts (#1461)
1 parent 7718a4d commit f25110c

File tree

1 file changed

+25
-60
lines changed

1 file changed

+25
-60
lines changed

.github/workflows/main.yml

Lines changed: 25 additions & 60 deletions
Original file line numberDiff line numberDiff line change
@@ -43,12 +43,12 @@ jobs:
4343

4444
steps:
4545
- name: Check out the repository
46-
uses: actions/checkout@v3
46+
uses: actions/checkout@v4
4747
with:
4848
persist-credentials: false
4949

5050
- name: Set up Python
51-
uses: actions/setup-python@v4
51+
uses: actions/setup-python@v5
5252
with:
5353
python-version: '3.9'
5454

@@ -79,12 +79,12 @@ jobs:
7979

8080
steps:
8181
- name: Check out the repository
82-
uses: actions/checkout@v3
82+
uses: actions/checkout@v4
8383
with:
8484
persist-credentials: false
8585

8686
- name: Set up Python ${{ matrix.python-version }}
87-
uses: actions/setup-python@v4
87+
uses: actions/setup-python@v5
8888
with:
8989
python-version: ${{ matrix.python-version }}
9090

@@ -104,24 +104,25 @@ jobs:
104104
run: |
105105
echo "date=$(date +'%Y-%m-%dT%H_%M_%S')" >> $GITHUB_OUTPUT #no colons allowed for artifacts
106106
107-
build:
108-
name: build packages
109-
110-
runs-on: ubuntu-latest
111-
112-
outputs:
113-
is_alpha: ${{ steps.check-is-alpha.outputs.is_alpha }}
114-
107+
test-build:
108+
name: verify packages / python ${{ matrix.python-version }} / ${{ matrix.os }} / ${{ matrix.dist-type }}
109+
runs-on: ${{ matrix.os }}
110+
strategy:
111+
fail-fast: false
112+
matrix:
113+
os: [ubuntu-latest, macos-14, windows-latest]
114+
python-version: ['3.9', '3.10', '3.11']
115+
dist-type: ["whl", "gz"]
115116
steps:
116117
- name: Check out the repository
117-
uses: actions/checkout@v3
118+
uses: actions/checkout@v4
118119
with:
119120
persist-credentials: false
120121

121-
- name: Set up Python
122-
uses: actions/setup-python@v4
122+
- name: Set up Python ${{ matrix.python-version }}
123+
uses: actions/setup-python@v5
123124
with:
124-
python-version: '3.9'
125+
python-version: ${{ matrix.python-version }}
125126

126127
- name: Install python dependencies
127128
run: |
@@ -136,12 +137,10 @@ jobs:
136137
run: ls -lh dist/
137138

138139
- name: Check distribution descriptions
139-
run: |
140-
twine check dist/*
140+
run: twine check dist/*
141141

142142
- name: Check wheel contents
143-
run: |
144-
check-wheel-contents dist/*.whl --ignore W007,W008
143+
run: check-wheel-contents dist/*.whl --ignore W007,W008
145144

146145
- name: Check if this is an alpha version
147146
id: check-is-alpha
@@ -150,44 +149,10 @@ jobs:
150149
if [[ "$(ls -lh dist/)" == *"a1"* ]]; then export is_alpha=1; fi
151150
echo "is_alpha=$is_alpha" >> $GITHUB_OUTPUT
152151
153-
test-build:
154-
name: verify packages / python ${{ matrix.python-version }} / ${{ matrix.os }}
155-
156-
if: needs.build.outputs.is_alpha == 0
157-
158-
needs: build
159-
160-
runs-on: ${{ matrix.os }}
161-
162-
strategy:
163-
fail-fast: false
164-
matrix:
165-
os: [ubuntu-latest, macos-14, windows-latest]
166-
python-version: ['3.9', '3.10', '3.11']
152+
- name: Install ${{ matrix.dist-type }} distributions
153+
if: ${{ steps.check-is-alpha.outputs.is_alpha == 0 }}
154+
run: find ./dist/*.${{ matrix.dist-type }} -maxdepth 1 -type f | xargs python -m pip install --force-reinstall --find-links=dist/
167155

168-
steps:
169-
- name: Set up Python ${{ matrix.python-version }}
170-
uses: actions/setup-python@v4
171-
with:
172-
python-version: ${{ matrix.python-version }}
173-
- name: Install python dependencies
174-
run: |
175-
python -m pip install --user --upgrade pip
176-
python -m pip install --upgrade wheel setuptools twine check-wheel-contents
177-
python -m pip --version
178-
179-
- name: Show distributions
180-
run: ls -lh dist/
181-
182-
- name: Install wheel distributions
183-
run: |
184-
find ./dist/*.whl -maxdepth 1 -type f | xargs python -m pip install --force-reinstall --find-links=dist/
185-
- name: Check wheel distributions
186-
run: |
187-
dbt --version
188-
- name: Install source distributions
189-
run: |
190-
find ./dist/*.gz -maxdepth 1 -type f | xargs python -m pip install --force-reinstall --find-links=dist/
191-
- name: Check source distributions
192-
run: |
193-
dbt --version
156+
- name: Check ${{ matrix.dist-type }} distributions
157+
if: ${{ steps.check-is-alpha.outputs.is_alpha == 0 }}
158+
run: python -c "import dbt.adapters.bigquery"

0 commit comments

Comments
 (0)