Skip to content

Align imports with Black/isort and add isort config - #12

Merged
dfeen87 merged 2 commits into
mainfrom
codex/fix-black-formatting-violations
Feb 10, 2026
Merged

Align imports with Black/isort and add isort config#12
dfeen87 merged 2 commits into
mainfrom
codex/fix-black-formatting-violations

Conversation

@dfeen87

@dfeen87 dfeen87 commented Feb 10, 2026

Copy link
Copy Markdown
Owner

Motivation

  • Ensure import formatting satisfies both black and isort style checks so CI/style checks pass.
  • Preserve runtime behavior and existing # noqa markers while normalizing import layout across modules.

Description

  • Add a .isort.cfg with profile = black to make isort follow Black's import formatting conventions.
  • Reformat imports in experiments/__init__.py, experiments/leggett_garg/__init__.py, experiments/process_tensor/__init__.py, experiments/reset_tests/__init__.py, simulations/__init__.py, simulations/markov_comparison.py, and theory/__init__.py to match Black/isort expectations without changing exported symbols.
  • Adjust simulations/markov_comparison.py to split the theory.dynamics import so the # noqa: E402 comment remains correctly associated while satisfying isort/black formatting rules.

Testing

  • Ran isort --check-only --diff theory/ analysis/ simulations/ experiments/ tests/ and iteratively fixed import ordering until isort --check-only --diff ... returned no errors.
  • Ran black --check theory/ analysis/ simulations/ experiments/ tests/ which reported no remaining formatting violations.
  • Observed black initially reformatted the seven affected files and subsequent black --check showed the files were stable.

Codex Task

@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.

@dfeen87
dfeen87 merged commit a01625d into main Feb 10, 2026
6 of 7 checks passed
@dfeen87
dfeen87 deleted the codex/fix-black-formatting-violations branch February 10, 2026 00:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant