Repository navigation
Fix CI lint failures by aligning Ruff rules with FastAPI DI and tightening auth error handling - #1
Conversation
|
Important Review skippedBot user detected. To trigger a single review, invoke the ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Advanced Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Co-authored-by: HMZ-H <119335393+HMZ-H@users.noreply.github.com>
HMZ-H
left a comment
There was a problem hiding this comment.
Fix CI lint failures by aligning Ruff rules with FastAPI DI and tightening auth error handling
|
@copilot resolve the merge conflicts in this pull request |
# Conflicts: # backend/dwf/api/deps.py Co-authored-by: HMZ-H <119335393+HMZ-H@users.noreply.github.com>
Resolved by merging |
The CI
lintjob failed due to Ruff violations introduced with the auth/migration changes, including import/order/type-style issues and bugbear checks conflicting with FastAPI dependency injection signatures.Lint policy alignment
Depends(...)calls as immutable defaults, preventing false-positiveB008on route dependencies.backend/pyproject.tomlundertool.ruff.lint.flake8-bugbear.Auth router exception correctness
except Exception.Codebase-wide Ruff conformance in touched backend paths