Skip to content

TST: Added test to check if _funcs.__all__ is in sync#828

Open
prady0t wants to merge 1 commit into
data-apis:mainfrom
prady0t:update_funcs.all
Open

TST: Added test to check if _funcs.__all__ is in sync#828
prady0t wants to merge 1 commit into
data-apis:mainfrom
prady0t:update_funcs.all

Conversation

@prady0t

@prady0t prady0t commented Jul 2, 2026

Copy link
Copy Markdown
Contributor

Closes: #821

Signed-off-by: Pradyot Ranjan <99216956+pradyotRanjan@users.noreply.github.com>
@prady0t

prady0t commented Jul 2, 2026

Copy link
Copy Markdown
Contributor Author

The failing test is already a great demonstration lol. @lucascolley have a look before I merge main.

@lucascolley

Copy link
Copy Markdown
Member

nice 👍 I wonder whether there is a way to get ruff to enforce this, though?

@prady0t

prady0t commented Jul 2, 2026

Copy link
Copy Markdown
Contributor Author

Ruff has checks for invalid entry in __all__, like :https://docs.astral.sh/ruff/rules/invalid-all-object/#why-is-this-bad
Or maybe checks for alphabetical order, duplicate entry, etc. Nothing that solves exactly this problem.

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.

MAINT: update _funcs.__all__

2 participants