Skip to content

feat: Check for __all__ presence in __init__.py files#766

Open
AcquaDiGiorgio wants to merge 3 commits intoDIRACGrid:mainfrom
AcquaDiGiorgio:issue-426-__all__-in-__init__
Open

feat: Check for __all__ presence in __init__.py files#766
AcquaDiGiorgio wants to merge 3 commits intoDIRACGrid:mainfrom
AcquaDiGiorgio:issue-426-__all__-in-__init__

Conversation

@AcquaDiGiorgio
Copy link

See #426

I tried something more sofisticated in python using importlib to check the variables in the module, both instantiated and imported, but there were multiple situations that were extremely difficult to check.

We also might want to set a common way of defining this variable, using either tuples or lists:

  • diracx/routers/health/__init__.py: __all__ = ["router"]
  • diracx/db/os/__init__.py __all__ = ("JobParametersDB",)

@read-the-docs-community
Copy link

read-the-docs-community bot commented Feb 5, 2026

Documentation build overview

📚 diracx | 🛠️ Build #31283317 | 📁 Comparing 336f514 against latest (5bba462)


🔍 Preview build

No files changed.

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.

1 participant