Remove unused imports and fix re-exports (F401)#556
Conversation
|
Thanks for the PR. Let me think about this for a bit - I'm not sure if I want to commit to the duplicate |
|
In my opinion, good practice is to be explicit about imports, and making sure those imports are used. Lines of code does not matter if it can make the parts clearer. To simply use |
|
Done, added re-exports for VideoStreamCv2, VideoStreamAv, and VideoStreamMoviePy. |
|
Last ask, can you rebase + update to export the new items I added for v0.7.1? Thanks for the PR BTW. |
ab2d6f2 to
c1fcb79
Compare
|
Rebased onto latest upstream main and added as re-exports for SourceSpan and VideoStreamConcat (the new v0.7.1 items). No problem! |
Remove unused imports across the codebase flagged by Ruff rule F401. Uses explicit re-exports in init.py to preserve the public API.