Skip to content

Conversation

@samay2504
Copy link
Contributor

No description provided.

@max-sixty
Copy link
Collaborator

ref #10843

@@ -0,0 +1,78 @@
"""Test for issue #10843: CombinedLock.locked() always returns true."""
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

can we have this as a test in a more idomatic place? ty!

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

done

@samay2504 samay2504 force-pushed the fix/xarray-10843-combined-lock-locked branch from ab0333a to 6f7885f Compare December 16, 2025 11:50
@garg-khushi
Copy link

Verified locally.
Checked out this branch and ran the backend lock tests:
python -m pytest -o addopts= xarray/tests/test_backends_locks.py
All tests pass. The change to call lock.locked() instead of accessing it as a property works as expected on my machine.

@max-sixty max-sixty merged commit 018ad08 into pydata:main Jan 4, 2026
40 checks passed
@welcome
Copy link

welcome bot commented Jan 4, 2026

Congratulations on completing your first pull request! Welcome to Xarray! We are proud of you, and hope to see you again! celebration gif

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants