Skip to content

Conversation

@kanterov
Copy link
Collaborator

Changes

Ensure that __all__ is sorted through ruff. See https://docs.astral.sh/ruff/rules/unsorted-dunder-all/

Why

Use a common convention for __all__ to make code more readable and idiomatic.

@kanterov kanterov temporarily deployed to test-trigger-is August 29, 2025 12:22 — with GitHub Actions Inactive
@kanterov kanterov marked this pull request as ready for review August 29, 2025 12:22
@eng-dev-ecosystem-bot
Copy link
Collaborator

eng-dev-ecosystem-bot commented Aug 29, 2025

Run: 17323691729

Env ✅​pass 🙈​skip
✅​ aws linux 308 504
✅​ aws windows 309 503
✅​ aws-ucws linux 421 401
✅​ aws-ucws windows 422 400
✅​ azure linux 308 503
✅​ azure windows 309 502
✅​ azure-ucws linux 421 400
✅​ azure-ucws windows 422 399
✅​ gcp linux 307 505
✅​ gcp windows 308 504

@kanterov kanterov added this pull request to the merge queue Aug 29, 2025
github-merge-queue bot pushed a commit that referenced this pull request Aug 29, 2025
## Changes
Ensure that `__all__` is sorted through ruff. See
https://docs.astral.sh/ruff/rules/unsorted-dunder-all/

## Why
Use a common convention for `__all__` to make code more readable and
idiomatic.
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Aug 29, 2025
@kanterov kanterov added this pull request to the merge queue Aug 29, 2025
Merged via the queue into main with commit 7c37dad Aug 29, 2025
19 checks passed
@kanterov kanterov deleted the sort-all branch August 29, 2025 15:55
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.

4 participants