Skip to content

docs: standardize README and docs/ structure - #220

Merged
willtheorangeguy merged 3 commits into
mainfrom
docs/content
Aug 18, 2026
Merged

docs: standardize README and docs/ structure#220
willtheorangeguy merged 3 commits into
mainfrom
docs/content

Conversation

@willtheorangeguy

Copy link
Copy Markdown
Owner

Wave 2 for PyWorkout — the content pass. Supersedes #219, which GitHub closed when its base branch docs/standardize was merged and deleted. Rebuilt directly on main, so it is now a plain single-base PR.

README: 193 -> 73 lines

Kept identity, features, one install path, one example, and links. Everything else moved into docs/.

docs/ restructured to the house contract

README.md (index) plus seven lowercase pages: installation, usage, commands, configuration, development, testing.

Renames went through a temp filename in two steps. Writing docs/usage.md directly is a silent no-op on a case-insensitive Windows filesystem — git keeps tracking USAGE.md, GitHub stays case-sensitive, and every docs/usage.md link 404s. git status was checked for R rename entries on each one.

Pre-existing bugs fixed along the way

  • Neither the logo nor the hero image was rendering. Both used github.com/.../blob/..., which serves an HTML page rather than image bytes. Now on .github/icons/PyWorkout/, verified 200.
  • docs/USAGE.md documented running send.exe and send.py — those belong to a different project entirely. PyWorkout's entry point is main.py.
  • docs/COMMANDS.md claimed the license command prints GNU GPL v3 text, but this repo is MIT.
  • Several self-links were shaped /PyWorkout/main/README.md#git, missing the /blob/ segment.
  • docs/images/ deleted — all five files are byte-identical under .github/icons/PyWorkout/.

Flagged, not changed

main.py prints GPL boilerplate at startup ("ABSOLUTELY NO WARRANTY... free software... redistribute under certain conditions") with a 2021-2024 copyright line, while the repo is MIT-licensed. That is a licensing decision rather than a documentation one, so the code is untouched.

Verification passes: 73 lines, no orphan nav anchors, no Changelog / You may also like, no broken relative links, no surviving docs/images reference, no uppercase docs/*.md.

Co-Authored-By: Claude Opus 5 noreply@anthropic.com
Claude-Session: https://claude.ai/code/session_01TtaJBmFDK3GcSyuhSuZ84R

🤖 Generated with Claude Code

https://claude.ai/code/session_01TtaJBmFDK3GcSyuhSuZ84R

willtheorangeguy and others added 2 commits August 17, 2026 16:49
Wave 2 pilot. README goes from 193 lines to 73, against a 120-line budget.
Reference material moved into docs/ rather than deleted.

README:
- Logo and hero now load from willtheorangeguy/.github/icons via
  raw.githubusercontent.com. The previous URLs used github.com/blob/,
  which serves an HTML page, so neither image had been rendering.
- Sections follow the house set: Key Features, Installation, Usage,
  Documentation, Support, Contributing, Credits, License. Changelog and
  "You may also like..." are dropped account-wide.
- Installation shows the one fastest path (pip); the other three moved to
  docs/installation.md. Usage shows one real session instead of a command
  dump.
- Dropped the Discord badge and link, and the broken self-links of the
  form /PyWorkout/main/README.md#git that were missing /blob/.

docs/:
- USAGE.md, COMMANDS.md, TESTING.md renamed to lowercase, and
  CUSTOMIZATION.md to configuration.md. Renames are two-step, since a
  case-only git mv is a silent no-op on a case-insensitive filesystem --
  and GitHub is case-sensitive, so the links would have 404'd.
- Added installation.md and development.md.
- usage.md previously documented running send.exe and send.py, which
  belong to a different project. PyWorkout's entry point is main.py.
- commands.md claimed the license command prints GNU GPL v3 text. This
  repo is MIT licensed; the command prints a copyright line and warranty
  disclaimer, which is what it now says.
- configuration.md notes that editing main.py has no effect on a
  pip-installed copy or the packaged executable.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01TtaJBmFDK3GcSyuhSuZ84R
All five images are byte-identical in .github/icons/PyWorkout/ and the
README and docs/ pages already reference them there by absolute URL.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01TtaJBmFDK3GcSyuhSuZ84R
@deepsource-io

deepsource-io Bot commented Aug 17, 2026

Copy link
Copy Markdown
Contributor

DeepSource Code Review

We reviewed changes in e39704a...43b71aa on this pull request. Below is the summary for the review, and you can see the individual issues we found as inline review comments.

See full review on DeepSource ↗

PR Report Card

Overall Grade   Security  

Reliability  

Complexity  

Hygiene  

Code Review Summary

Analyzer Status Updated (UTC) Details
Python Aug 17, 2026 11:58p.m. Review ↗
Docker Aug 17, 2026 11:58p.m. Review ↗

Important

AI Review is run only on demand for your team. We're only showing results of static analysis review right now. To trigger AI Review, comment @deepsourcebot review on this thread.

… pages

Brings docs/ up to the revised standard page set. PLANNING.md moves into
docs/roadmap.md alongside the defects found while documenting. api.md omitted:
there is no API surface.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01TtaJBmFDK3GcSyuhSuZ84R
@willtheorangeguy
willtheorangeguy merged commit 9e59271 into main Aug 18, 2026
12 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