Skip to content

chore: publication aftercare (ruff format, format check in CI, banner) - #4

Merged
lukisch merged 2 commits into
mainfrom
chore/publish-aftercare
Oct 11, 2026
Merged

lukisch merged 2 commits into
mainfrom
chore/publish-aftercare

Conversation

@lukisch

@lukisch lukisch commented Oct 11, 2026

Copy link
Copy Markdown
Contributor

What

Housekeeping before the repository is opened up. No behaviour change.

  1. style: apply ruff format to the whole tree (a4d71a3): 26 files reformatted with ruff 0.17.0.
    The AST of every changed Python file is identical before and after (ast.dump comparison).
  2. docs: publication aftercare (320e28d):
    • README (EN/DE) banner assets/banner.png, vector source assets/banner.svg (drawn in code, no third-party marks)
    • CI lint step additionally runs ruff format --check .; README and CONTRIBUTING list it
    • one test comment worded neutrally
    • CHANGELOG [Unreleased] entries

Checks (local, Windows, Python 3.12, fresh venv)

  • ruff check .: clean; ruff format --check .: 38 files already formatted
  • python -m pytest -q: 145 passed, 1 skipped (same as main before the change)
  • GITHUBBOT_CORE_CONSUMER_TEST=1 python -m pytest -q tests/test_consumer_install.py: 1 passed
  • python -m build: sdist and wheel build; assets/ is not part of either artifact
  • gitleaks 8.30.1 over all refs and the working tree: no leaks

Note for the release decision

The wheel content changes (reformatted sources, same AST). Whether this warrants a v0.2.2 patch tag is left to the reviewer/maintainer; the version in __init__ is unchanged (0.2.1).

Draft until reviewed by a different model. Do not merge without that review.

🤖 Generated with Claude Code

https://claude.ai/code/session_01LYHP4Js5ggpzQ78JXWRe5E

Lukas Geiger and others added 2 commits October 11, 2026 20:38
26 files reformatted with ruff 0.17.0; no code change. Verified: the AST of
every changed Python file is identical before and after (ast.dump compare),
ruff check clean, pytest 145 passed / 1 skipped.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01LYHP4Js5ggpzQ78JXWRe5E
… comment)

- README (EN/DE): banner assets/banner.png (vector source assets/banner.svg,
  drawn in code, no third-party marks)
- CI lint step also runs `ruff format --check .`; README/CONTRIBUTING list it
- test_system_agent: pinned-hash comment worded neutrally
- CHANGELOG: Unreleased entries

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01LYHP4Js5ggpzQ78JXWRe5E
@lukisch
lukisch marked this pull request as ready for review October 11, 2026 19:14
@lukisch
lukisch merged commit 2ad5843 into main Oct 11, 2026
2 checks passed
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