Skip to content

Upgrade black Python formatting - #7533

Merged
nilsdeppe merged 2 commits into
sxs-collaboration:developfrom
nilsvu:upgrade_black
Sep 4, 2026
Merged

Upgrade black Python formatting#7533
nilsdeppe merged 2 commits into
sxs-collaboration:developfrom
nilsvu:upgrade_black

Conversation

@nilsvu

@nilsvu nilsvu commented Sep 2, 2026

Copy link
Copy Markdown
Member

Proposed changes

Upgrade instructions

Rerun pip install -r support/Python/dev_requirements.txt in your Python env (or enable SPECTRE_FETCH_MISSING_DEPS) to upgrade your installed black version.

Code review checklist

  • The code is documented and the documentation renders correctly. Run
    make doc to generate the documentation locally into BUILD_DIR/docs/html.
    Then open index.html.
  • The code follows the stylistic and code quality guidelines listed in the
    code review guide.
  • The PR lists upgrade instructions and is labeled bugfix or
    new feature if appropriate.
  • If a coding agent was used, have a co-author trailer of the form
    "Co-Authored-By: " as the last line of the
    commit, e.g. "Co-Authored-By: Claude Sonnet 4.6 noreply@anthropic.com",
    "Co-Authored-by: Codex noreply@openai.com", or
    "Co-Authored-By: GitHub Copilot CLI noreply@microsoft.com".

Further comments

nilsvu and others added 2 commits September 3, 2026 00:25
The build system already requires it, but `setup.cfg` still declared `>=3.8`
and black still targeted py37.

That makes the Python 2 branches dead. In Pypp they also contained
`static_assert(false, ...)` in the `#else` branches, several of them missing
a semicolon.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
@nilsdeppe
nilsdeppe merged commit d68dbb0 into sxs-collaboration:develop Sep 4, 2026
44 of 46 checks passed
@nilsvu
nilsvu deleted the upgrade_black branch September 4, 2026 15:45
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.

3 participants