Skip to content

fix(ci): upgrade typing to 3.10+ style, fix isort CI failures (#2608)#2662

Merged
mrveiss merged 1 commit intoDev_new_guifrom
fix/issue-2608-isort-ci
Mar 27, 2026
Merged

fix(ci): upgrade typing to 3.10+ style, fix isort CI failures (#2608)#2662
mrveiss merged 1 commit intoDev_new_guifrom
fix/issue-2608-isort-ci

Conversation

@mrveiss
Copy link
Copy Markdown
Owner

@mrveiss mrveiss commented Mar 27, 2026

Summary

  • Upgrades Python typing syntax to 3.10+ (Dictdict, Optional[X]X | None, etc.) in 15 files causing CI isort failures
  • Adds from __future__ import annotations to all affected files
  • Root cause: isort 8.0.1 classifies imports differently on Python 3.10 vs 3.12 AST

Test plan

  • CI code-quality check passes (isort + black + flake8)
  • All pre-commit hooks pass locally

Closes #2608

)

CI code-quality was failing on every PR due to 15 files with isort
errors caused by Python 3.10 vs 3.12 AST parsing differences.

Fix: add `from __future__ import annotations` and modernize typing
(Dict→dict, List→list, Optional[X]→X|None, Tuple→tuple) in all 15
failing files, eliminating the isort divergence between environments.
@mrveiss mrveiss merged commit a199004 into Dev_new_gui Mar 27, 2026
3 of 4 checks passed
@github-actions
Copy link
Copy Markdown

⚠️ SSOT Configuration Compliance: Violations Found

Metric Count
Total Violations 2
SSOT Violations (high priority) 1
Other Violations 1

⚠️ 1 values have SSOT config equivalents!

These should be replaced with SSOT config imports:

Python:

from src.config.ssot_config import config
# Use: config.vm.main, config.port.backend, config.backend_url

TypeScript:

import config from '@/config/ssot-config'
// Use: config.vm.main, config.port.backend, config.backendUrl

📖 See SSOT_CONFIG_GUIDE.md for documentation.

@mrveiss mrveiss deleted the fix/issue-2608-isort-ci branch March 27, 2026 22:39
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.

1 participant