docs: fix grammatical and readability problems README.md - #1017
docs: fix grammatical and readability problems README.md#1017xHope-tech wants to merge 7 commits into
Conversation
Corrected capitalization of 'Flatpak' in multiple sections of the README.
…fic Configuration Add missing ":" under ### system_files/bluefin/ - Bluefin Specific Configuration: "Files specific to Bluefin"
Corrected punctuation by adding periods to list items in the README. docs: add missing periods (.) in README.md.
hanthor
left a comment
There was a problem hiding this comment.
Read the full diff — purely cosmetic README prose fixes: capitalizing "Flatpak" consistently and adding missing trailing periods to bullet-list items throughout. No links, code, commands, or documented behavior changed, and no line's meaning shifted. Single file (README.md).
No CI has run yet (likely pending a maintainer's one-time workflow-run approval for a first-time external contributor), but that's not a blocker here — the checks that exist (validate.yml, build.yml, skill-drift.yml, pr-e2e.yml) test shell/build/image behavior, none of which this PR touches.
Generated by Claude Code
There was a problem hiding this comment.
Approved by @castrojo for Hive auto-merge on green CI.
hanthor
left a comment
There was a problem hiding this comment.
Reads clean and merges clean — one factual nit you could fix while you're in here
Verified against current main: merges without conflict, README-only (38 insertions / 38 deletions, no other file), so nothing under system_files/ and no bluefin / bluefin-lts / dakota blast radius. The flatpak → Flatpak capitalisation is right — it is a proper noun, and the repo was inconsistent about it.
Nit, pre-existing and not introduced by you. Two lines you are already touching make a count claim that is now off by one:
- **`system-flatpaks.Brewfile`** - Core Flatpaks installed on all Bluefin variants (37 applications ...
$ grep -cE '^[[:space:]]*flatpak ' system_files/bluefin/usr/share/ublue-os/homebrew/system-flatpaks.Brewfile
38
$ grep -cE '^[[:space:]]*flatpak ' system_files/bluefin/usr/share/ublue-os/homebrew/system-dx-flatpaks.Brewfile
6
The DX count (6) is still correct; the core count is 38, not 37. Since your diff already rewrites that exact line, bumping 37 → 38 is a free fix. Worth considering dropping the hardcoded number entirely — it is guaranteed to drift again and nothing in CI checks it.
Second nit on the same line, also pre-existing: it says "installed on all Bluefin variants", but the file lives under system_files/bluefin/, not system_files/shared/, so it does not reach bluefin-lts or dakota. "on Bluefin" would be more accurate. Your call whether that is in scope for a grammar pass.
On the checklist: just check and pre-commit run --all-files are unticked. Neither can be affected by a README-only change (just check only formats *.just and the Justfile), so that is not a real gap — but note there are currently zero reported checks on head 90c6a567:
$ pull_request_read get_status 1017
{"state":"pending","sha":"90c6a567...","total_count":0,"statuses":[]}
mergeable_state is blocked, so this is waiting on a maintainer review plus a check actually reporting, not on anything in the diff.
Generated by Claude Code
bluefin-common PR
What does this change?
Fix grammatical and readability problems README.md. Under ### system_files/bluefin/ - Bluefin Specific Configuration. Correct capitalization of 'Flatpak' in multiple sections of the README. Add missing periods (.) in multiple sections of the README.md.
Why?
Fix grammatical and readability problems README.md. Under ### system_files/bluefin/ - Bluefin Specific Configuration. Correct capitalization of 'Flatpak' in multiple sections of the README. Add missing periods (.) in multiple sections of the README.md.
Closes #
PR pipeline
Checklist
fix:,feat:,docs:,ci:,refactor:, etc.)just checkpassespre-commit run --all-filespassesdocs/skills/skill-improvement.md)AGENTS.md/docs/SKILL.md/docs/skills/links remain validgh run list --repo projectbluefin/common --limit 5AI attribution
If this PR includes AI-authored commits, include both trailers: