Skip to content

Commit d4f09d6

Browse files
committed
nits
1 parent 3d0567f commit d4f09d6

File tree

2 files changed

+11
-4
lines changed

2 files changed

+11
-4
lines changed

.github/workflows/ci.yml

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -51,7 +51,12 @@ jobs:
5151
strategy:
5252
fail-fast: false
5353
matrix:
54-
environment: [tests-py310, tests-py313, tests-numpy1, tests-backends, tests-nogil]
54+
environment:
55+
- tests-py310
56+
- tests-py313
57+
- tests-numpy1
58+
- tests-backends
59+
- tests-nogil
5560
runs-on: [ubuntu-latest]
5661

5762
steps:
@@ -66,6 +71,7 @@ jobs:
6671
environments: ${{ matrix.environment }}
6772

6873
- name: Test package
74+
# Save some time; also at the moment of writing coverage crashes on python 3.13t
6975
if: ${{ matrix.environment != 'tests-nogil' }}
7076
run: pixi run -e ${{ matrix.environment }} tests-ci
7177

src/array_api_extra/testing.py

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -264,10 +264,11 @@ def xp(request):
264264
265265
Notes
266266
-----
267-
This context manager is thread unsafe on Dask and JAX. If you run your test suite
268-
with
267+
This context manager monkey-patches modules and as such is thread unsafe
268+
on Dask and JAX. If you run your test suite with
269269
`pytest-run-parallel <https://github.com/Quansight-Labs/pytest-run-parallel/>`_,
270-
you should mark the test or the fixture with ``@pytest.mark.thread_unsafe``.
270+
you should mark these backends with ``@pytest.mark.thread_unsafe``, as shown in
271+
the example above.
271272
"""
272273
mod = cast(ModuleType, request.module)
273274
mods = [mod, *cast(list[ModuleType], getattr(mod, "lazy_xp_modules", []))]

0 commit comments

Comments
 (0)