Skip to content

Add an optional xrootd extra and use it in CI#117

Merged
lobis merged 7 commits into
scikit-hep:mainfrom
lobis:add-optional-xrootd-extra
Apr 22, 2026
Merged

Add an optional xrootd extra and use it in CI#117
lobis merged 7 commits into
scikit-hep:mainfrom
lobis:add-optional-xrootd-extra

Conversation

@lobis
Copy link
Copy Markdown
Collaborator

@lobis lobis commented Apr 21, 2026

Summary

  • add a dedicated xrootd optional dependency extra in pyproject.toml
  • update nox, docs, and contributor install instructions to opt into that extra where needed
  • switch CI/docs workflows from manual mamba install xrootd setup to pip install with the extra
  • lazily import the XRootD-backed classes so import fsspec_xrootd still works without the extra and points users to the install hint when accessed

Testing

  • ./.venv39/bin/python -m pytest tests/test_package.py
  • python3 -m compileall src tests
  • isolated python3 -m venv install of -e . without xrootd, verifying import fsspec_xrootd still works and accessing XRootDFileSystem raises the new guidance

Refs #89

@lobis
Copy link
Copy Markdown
Collaborator Author

lobis commented Apr 21, 2026

@lgray feel free to comment on the discussion at #89

@lobis lobis merged commit 19edfad into scikit-hep:main Apr 22, 2026
17 checks passed
@lobis lobis deleted the add-optional-xrootd-extra branch April 22, 2026 09:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants