Skip to content

chore(#489): drop stale Python 3.8/3.9 classifiers from pyproject.toml#555

Merged
drewdrewthis merged 1 commit into
mainfrom
fix/issue489-stale-classifiers
May 29, 2026
Merged

chore(#489): drop stale Python 3.8/3.9 classifiers from pyproject.toml#555
drewdrewthis merged 1 commit into
mainfrom
fix/issue489-stale-classifiers

Conversation

@drewdrewthis

Copy link
Copy Markdown
Collaborator

Summary

  • Removes Programming Language :: Python :: 3.8 and Programming Language :: Python :: 3.9 from the classifier block in python/pyproject.toml.
  • requires-python = ">=3.10" already declares the minimum; the stale 3.8/3.9 entries caused false-positive PyPI search results.
  • No doc files with the stale 68 @unit counts were found in-repo (they were never committed or were already absent).

Acceptance criteria (from #489)

  • python/pyproject.toml classifier block drops "Programming Language :: Python :: 3.8" and "3.9" entries.
  • Any in-repo doc referencing "68 @Unit" updated — grep found zero hits; no files to change.
  • docs/proposals/issue-350-prove-it-report.md is absent from the repo (was only on @drewdrewthis's local checkout per the audit report reference); no change needed.

Test plan

  • CI passes (Python classifiers are metadata; no runtime behaviour changed)
  • python/pyproject.toml contains no 3.8 or 3.9 classifier lines

🤖 Generated with Claude Code

requires-python = ">=3.10" but classifiers advertised 3.8 and 3.9,
causing false-positive PyPI search results. Remove the two stale
classifier lines so the declared supported versions match the runtime
constraint.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@drewdrewthis drewdrewthis added the grinding Grinder is actively managing this PR label May 25, 2026
@github-actions github-actions Bot added the low-risk-change PR qualifies as low-risk per policy and can be merged without manual review label May 25, 2026
@github-actions

Copy link
Copy Markdown
Contributor

Automated low-risk assessment

This PR was evaluated against the repository's Low-Risk Pull Requests procedure.

  • Scope: Remove "Programming Language :: Python :: 3.8" and "... :: 3.9" classifier entries from python/pyproject.toml (no other files modified).
  • Exclusions confirmed: no changes to auth, security settings, database schema, business-critical logic, or external integrations.
  • Classification: low-risk-change under the documented policy.

The change only removes stale PyPI trove classifier lines (Python 3.8 and 3.9) from python/pyproject.toml and does not modify authentication, secrets, database schemas, business‑critical logic, or third‑party integrations. Runtime requirements are unchanged (requires-python remains >=3.10), so this is metadata-only and easy to revert if needed.

An approving review has been submitted by automation. The PR may merge once required CI checks pass.

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approved by automation: PR qualifies as low-risk-change under the documented policy.

@drewdrewthis

Copy link
Copy Markdown
Collaborator Author

[grinder] READY for human review

CI: green (zero failing, zero pending)
ACs: met — Python 3.8 and 3.9 classifier entries removed from python/pyproject.toml, aligning advertised versions with requires-python = ">=3.10"; no stale doc files found in repo
Threads: zero unresolved

Verified by:
command gh pr checks 555 → all 13 checks pass (test 3.12: pass 5m52s, python-complete: pass, javascript-complete: pass)
command gh pr view 555 --json reviewDecision → APPROVED by github-actions (low-risk-change policy)

@drewdrewthis drewdrewthis added pr-ready and removed grinding Grinder is actively managing this PR labels May 25, 2026
@drewdrewthis drewdrewthis merged commit eb33452 into main May 29, 2026
23 of 29 checks passed
@drewdrewthis drewdrewthis deleted the fix/issue489-stale-classifiers branch May 29, 2026 12:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

low-risk-change PR qualifies as low-risk per policy and can be merged without manual review pr-ready

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant