Skip to content

chore: remove vestigial setup.cfg, add [tool.wads.ci], migrate CI to the stub - #22

Open
thorwhalen wants to merge 1 commit into
masterfrom
doctor/setup-cfg-and-wads-ci
Open

thorwhalen wants to merge 1 commit into
masterfrom
doctor/setup-cfg-and-wads-ci

Conversation

@thorwhalen

Copy link
Copy Markdown
Member

Summary

Found via a Priority E doctor-sweep pass (AI enablement / README-vs-code / wads-setup audit):

  • Removed setup.cfg. It was pure legacy cruft -- pyproject.toml has been the real SSOT for a while (hatchling backend, full [project] metadata matching), nothing in the repo referenced setup.cfg except a stale CI comment ("MIGRATION NOTE: Old CI references setup.cfg - ensure project is migrated to pyproject.toml first"), and its only activity was the CI bot's version-bump push-back keeping its version = in sync with a value nothing actually read.
  • Added [tool.wads.ci]. It was entirely absent even though ci.yml already reads it (silently falling back to the read-ci-config action's defaults) -- added project_name = "config2py" and testing.python_versions = ["3.10", "3.12"] explicitly.
  • Migrated CI to the reusable-workflow stub. Was on "Modern uv (inline)", CI green -> sanctioned moment per the wads-migrate skill.

Everything else checked clean: no .claude/ (fact, not staleness); README's "For AI agents" section (added by #20 last week) is accurate; config_getter, is_repl, and the codec/extras story all match the current source.

Config-only change; self-reviewed only.

Test plan

🤖 Generated with Claude Code

https://claude.ai/code/session_011HSBVhDjRU4apSLcRkavv9

…the stub

setup.cfg was pure legacy cruft: pyproject.toml has been the SSOT
(hatchling backend, full [project] metadata) for a while, nothing in
the repo referenced setup.cfg except a stale CI comment, and its only
activity was the CI bot's version-bump push-back keeping it in sync
with a value nothing reads. Removed.

[tool.wads.ci] was entirely absent even though ci.yml already reads
it -- added project_name + testing.python_versions explicitly. Then
converted the already-green Modern-uv-inline workflow to the reusable
stub (sanctioned per wads-migrate once CI is green).
@thorwhalen

Copy link
Copy Markdown
Member Author

Same action_required/zero-jobs anomaly as i2mint/py2mcp#15, thorwhalen/artful#11, thorwhalen/skill#10 (4th confirmation this session) -- hits every repo's FIRST-ever call to i2mint/wads/.github/workflows/uv-ci.yml@master, never a repo already on it. wads ci-local is fully green locally. Leaving this PR open; filed a manual-task issue.

Self-reviewed only (config-only change, no application code behaviour altered).

thorwhalen added a commit that referenced this pull request Sep 22, 2026
Module map, verified test/lint commands matching CI's actual doctest
flags, the DFLT_MASKING_INPUT secrets-echo gotcha (issue #13) and
pointers to #16/#22/#23, and dependents. Docs-only.

Co-authored-by: Claude Opus 5 <noreply@anthropic.com>
@thorwhalen

Copy link
Copy Markdown
Member Author

Verified from the API at the end of the 2026-09-22 crowsnest run: this run sits at action_required with zero jobs, POST /actions/runs/{id}/approve refuses it ("not from a fork pull request"), and rerun refuses it too. It needs the one-time approval button in this repo's Actions tab — the first call to the shared reusable workflow. wads ci-local is green on this branch; nothing else blocks the merge.

thorwhalen added a commit that referenced this pull request Sep 27, 2026
- license = "Apache-2.0" (PEP 639) plus license-files, replacing the
  deprecated [project.license] table; no License :: classifier.
- Classifiers for Python 3.10-3.13 (suite verified on each), keywords,
  author, and Documentation/Repository/Issues URLs (docs site checked live).
- build-system floor hatchling>=1.27, the first release with PEP 639 support.

Dependencies, version and CI are unchanged; setup.cfg removal and the CI
stub migration stay in #22. twine check passes on the sdist and wheel, and
the CI version-bump regex still targets [project].version only.

This branch has not been deployed

No deployments
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