Skip to content

Commit ac21520

Browse files
committed
Address comments for reusability
1 parent e8d7aed commit ac21520

2 files changed

Lines changed: 11 additions & 16 deletions

File tree

.github/workflows/build.yml

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -496,6 +496,12 @@ jobs:
496496
name: hypothesis-example-db
497497
path: ${{ env.CPYTHON_BUILDDIR }}/.hypothesis/examples/
498498

499+
test-lazy-imports-all:
500+
name: 'Test Lazy Imports Fully Enabled'
501+
needs: build-context
502+
if: fromJSON(needs.build-context.outputs.run-tests)
503+
uses: ./.github/workflows/reusable-lazy-imports-all.yml
504+
499505
build-asan:
500506
name: 'Address sanitizer'
501507
runs-on: ${{ matrix.os }}
@@ -669,6 +675,7 @@ jobs:
669675
- build-emscripten
670676
- build-wasi
671677
- test-hypothesis
678+
- test-lazy-imports-all
672679
- build-asan
673680
- build-san
674681
- cross-build-linux

.github/workflows/lazy-imports-all.yml renamed to .github/workflows/rusable-test-lazy-imports-all.yml

Lines changed: 4 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
name: Tests (Lazy Imports)
1+
name: Reusable Lazy Imports Tests
22

33
# Run the CPython test suite with global lazy imports forced on
44
# (``PYTHON_LAZY_IMPORTS=all``, equivalent to ``-X lazy_imports=all``).
@@ -9,19 +9,7 @@ name: Tests (Lazy Imports)
99
# against regressions.
1010

1111
on:
12-
workflow_dispatch:
13-
push:
14-
branches: &branches
15-
- 'main'
16-
- '3.*'
17-
paths-ignore: &paths-ignore
18-
- 'Doc/**'
19-
- 'Misc/**'
20-
- '**/*.md'
21-
- '**/*.rst'
22-
pull_request:
23-
branches: *branches
24-
paths-ignore: *paths-ignore
12+
workflow_call:
2513

2614
permissions:
2715
contents: read
@@ -34,8 +22,8 @@ env:
3422
FORCE_COLOR: 1
3523

3624
jobs:
37-
lazy-imports:
38-
name: lazy_imports=all enabled
25+
test-lazy-imports-all:
26+
name: 'Test Lazy Imports Fully Enabled'
3927
runs-on: ubuntu-24.04
4028
timeout-minutes: 60
4129
env:

0 commit comments

Comments
 (0)