Conversation
There was a problem hiding this comment.
Pull request overview
This PR updates pre-commit hook dependencies and applies Black formatting changes to improve code consistency across the codebase. The updates include upgrading pre-commit-hooks from v5.0.0 to v6.0.0 and Black from 25.1.0 to 26.1.0, along with automated formatting adjustments.
Changes:
- Updated pre-commit hook versions in
.pre-commit-config.yaml - Removed unnecessary parentheses from tuple unpacking assignments throughout the codebase (more Pythonic style)
- Consolidated multi-line function calls and string formatting to single lines where appropriate
- Cleaned up whitespace inconsistencies (removed extra blank lines, adjusted line breaks)
Reviewed changes
Copilot reviewed 16 out of 16 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| .pre-commit-config.yaml | Updated pre-commit-hooks to v6.0.0 and Black to v26.1.0 |
| python/tk_multi_loader/utils.py | Consolidated multi-line error logging statement |
| python/tk_multi_loader/model_latestpublish.py | Removed parentheses from tuple unpacking |
| python/tk_multi_loader/dialog.py | Removed parentheses from tuple unpacking and consolidated multi-line statements |
| python/tk_multi_loader/delegate_publish_thumb.py | Removed parentheses from tuple unpacking |
| python/tk_multi_loader/delegate_publish_list.py | Removed parentheses from tuple unpacking |
| python/tk_multi_loader/api/manager.py | Removed extra blank line |
| hooks/tk-shell_actions.py | Added blank line after docstring |
| hooks/tk-photoshop_actions.py | Added blank line after docstring |
| hooks/tk-nuke_actions.py | Added blank line after docstring and removed parentheses from tuple unpacking |
| hooks/tk-motionbuilder_actions.py | Added blank line after docstring and removed parentheses from tuple unpacking |
| hooks/tk-maya_actions.py | Consolidated multi-line function call |
| hooks/tk-mari_actions.py | Added blank line after docstring |
| hooks/tk-houdini_actions.py | Added blank line after docstring |
| hooks/tk-flame_actions.py | Added blank line after docstring |
| hooks/tk-3dsmaxplus_actions.py | Added blank line after docstring and removed parentheses from tuple unpacking |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #133 +/- ##
===========================================
- Coverage 77.09% 19.72% -57.38%
===========================================
Files 33 35 +2
Lines 2633 2794 +161
===========================================
- Hits 2030 551 -1479
- Misses 603 2243 +1640
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
carlos-villavicencio-adsk
left a comment
There was a problem hiding this comment.
I'd suggest to disable codecov so we don't block this task.
This pull request focuses on code quality improvements and dependency updates. The main changes include updating pre-commit hook dependencies, consistently unpacking tuples without parentheses for Pythonic style, and minor formatting adjustments throughout the codebase.