Skip to content

polish(docs): remove stale roadmap bullet, add PyPI classifiers - #291

Merged
TexasCoding merged 1 commit into
mainfrom
polish/272-docs
May 21, 2026
Merged

TexasCoding merged 1 commit into
mainfrom
polish/272-docs

Conversation

@TexasCoding

Copy link
Copy Markdown
Owner

Closes #272

Part of #273 (post-v2.4 audit), Wave 3.

- ROADMAP: drop redundant 'Unreleased (post-v2.2.0)' bullet (v2.3.0
  already listed below as shipped).
- pyproject: add Framework :: AsyncIO, Topic :: Office/Business ::
  Financial :: Investment, Operating System :: OS Independent
  classifiers for accurate PyPI faceted-search discovery.

Closes #272
@claude

claude Bot commented May 21, 2026

Copy link
Copy Markdown

Code Review — PR #291: polish(docs): remove stale roadmap bullet, add PyPI classifiers

Overview: A small, focused polish PR addressing two independent items from issue #272. Removes a stale ROADMAP entry and adds three missing PyPI trove classifiers to pyproject.toml. No functional code changes.


ROADMAP.md

  • The removed bullet (Unreleased (post-v2.2.0)) was genuinely confusing — it appeared directly under the v2.4.0 section, implying in-flight work that already shipped as v2.3.0. The deletion is correct and clean.
  • No adjacent content disrupted; v2.3.0's normal entry remains below. ✓

pyproject.toml — classifiers

All three additions are valid PyPI trove classifiers:

Classifier Accurate?
Framework :: AsyncIO ✓ — AsyncKalshiClient ships in the package
Operating System :: OS Independent ✓ — pure Python + httpx; no platform-specific code
Topic :: Office/Business :: Financial :: Investment ✓ — real-money prediction market trading SDK

Insertion order maintains the existing alphabetical sort, which is consistent style. ✓

One minor suggestion (non-blocking): Consider also adding Topic :: Software Development :: Libraries :: Python Modules — it's the standard classifier for SDK/library packages and increases visibility in the Software Development facet on PyPI. But this is entirely optional and could be deferred to a later wave.


Test coverage

No tests required — this PR touches only documentation and package metadata.


Overall

Clean, surgical, and accurate. Both changes are exactly as described in #272. No concerns.

Verdict: ✅ Approve — ready to merge.

@TexasCoding
TexasCoding merged commit 240a507 into main May 21, 2026
5 checks passed
@TexasCoding
TexasCoding deleted the polish/272-docs branch May 21, 2026 13:54
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.

Polish bundle: docs cleanup (stale ROADMAP entry, missing PyPI classifiers)

1 participant