chore: sort imports in vla_jepa tests - #4354
Merged
Merged
Conversation
Pre-existing `I001` violations that `pre-commit run --all-files` auto-fixes. They are unrelated to the documentation work but block the green run the rest of this branch is verified against; confirmed present on main before this branch. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
5 tasks
CarolinePascal
self-requested a review
August 7, 2026 13:22
CarolinePascal
approved these changes
Aug 7, 2026
Collaborator
|
We need to merge #4348 first for the CI to run correctly |
github-actions
Bot
added
the
tests
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary / Motivation
Three pre-existing
I001(unsorted imports) violations thatpre-commit run --all-filesauto-fixes. Confirmed present onmainbefore this change.Split out as a standalone PR so the six-PR documentation-infrastructure stack keeps a diff containing only documentation work.
Related issues
What changed
Import blocks sorted in three
tests/policies/vla_jepa/test files. No functional change — the fix is exactly what the repo's own ruff hook produces.How was this tested (or how to run locally)
pre-commit run --all-filesis green after this change and red before it.Checklist (required before merge)
pre-commit run -a)pytest)Reviewer notes
Trivial and mechanical; merge whenever convenient.