Skip to content

Add ruff linting and fix all 680 errors across codebase#1

Draft
Copilot wants to merge 2 commits into
mainfrom
copilot/analyze-code-for-linting-errors
Draft

Add ruff linting and fix all 680 errors across codebase#1
Copilot wants to merge 2 commits into
mainfrom
copilot/analyze-code-for-linting-errors

Conversation

Copy link
Copy Markdown

Copilot AI commented Mar 7, 2026

Full linting pass on the codebase using ruff. Added linter configuration and fixed all 680 errors found.

Configuration

  • Added [tool.ruff] to pyproject.toml: line-length = 120, rules E/W/F/I

Fixes by category

  • F401 (35): Removed unused imports (sys, json, os, unused symbols in try/except availability checks)
  • I001 (33): Sorted import blocks
  • F541 (28): Converted placeholder-less f-strings to regular strings
  • E501 (580): Reformatted via ruff format + manual wrapping of long string literals in sample generators
  • E741 (2): Renamed ambiguous loop variable llang/line
  • F841 (2): Removed unused variables (box_height, headings)

Verify

ruff check .   # All checks passed
ruff format .  # 62 files left unchanged

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

- 04_html: Break long HTML string with backslash continuation
- 05_spreadsheets: Use parenthesized string concatenation for descriptions
- 08_markdown_txt: Break long lines in triple-quoted strings with backslash continuation
- generate_presentation: Use parenthesized string concatenation for slide content
- Run ruff format for consistent style

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot AI changed the title [WIP] Analyze code for linting errors Add ruff linting and fix all 680 errors across codebase Mar 7, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants