[WEB-5044] fix: ruff lint and format errors#7868
Conversation
|
Linked to Plane Work Item(s) This comment was auto-generated by Plane |
|
Important Review skippedMore than 25% of the files skipped due to max files limit. The review is being skipped to prevent a low-quality review. 154 files out of 261 files are above the max files limit of 100. Please upgrade to Pro plan to get higher limits. You can disable this status message by setting the ✨ Finishing touches
🧪 Generate unit tests
Tip 👮 Agentic pre-merge checks are now available in preview!Pro plan users can now enable pre-merge checks in their settings to enforce checklists before merging PRs.
Please see the documentation for more information. Example: reviews:
pre_merge_checks:
custom_checks:
- name: "Undocumented Breaking Changes"
mode: "warning"
instructions: |
Pass/fail criteria: All breaking changes to public APIs, CLI flags, environment variables, configuration keys, database schemas, or HTTP/GraphQL endpoints must be documented in the "Breaking Change" section of the PR description and in CHANGELOG.md. Exclude purely internal or private changes (e.g., code not exported from package entry points or explicitly marked as internal).Please share your feedback with us on this Discord post. 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 |
There was a problem hiding this comment.
Pull Request Overview
This PR focuses on updating the Ruff linting configuration by increasing the line length limit from 88 to 120 characters and formatting the entire codebase to comply with the new settings. The changes primarily involve consolidating multi-line code onto single lines where appropriate and correcting formatting inconsistencies throughout the Python codebase.
- Increased Ruff line length from 88 to 120 characters in pyproject.toml
- Reformatted all Python files to comply with the new line length settings
- Fixed string quote consistency by converting single quotes to double quotes in multiple files
Reviewed Changes
Copilot reviewed 259 out of 261 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| apps/api/pyproject.toml | Updated line-length configuration from 88 to 120 |
| Multiple Python files | Reformatted code to fit new line length, consolidated multi-line statements |
| String literals | Converted single quotes to double quotes for consistency |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
* fix: lint errors * fix: file formatting * fix: code refactor
Description
Type of Change