docs: sharpen homepage pitch and drop duplicate visuals - #3
Merged
Merged
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #3 +/- ##
============================================
- Coverage 58.85% 58.46% -0.40%
Complexity 15 15
============================================
Files 22 22
Lines 1016 1016
Branches 230 230
============================================
- Hits 598 594 -4
- Misses 346 348 +2
- Partials 72 74 +2
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
- Lead with the tagline as the H1 (Code coverage; Typescript, iOS, and Android; no italics) and drop the repeated tagline line. - Reorder the pitch: install into a Pattern C test app first, then what the module/CLI does — flush, merge every framework .profraw + app binary into LCOV/Jacoco, remap instrumented JS line-for-line to TS. - Reword the CI angle to "use it as a signal in your dev loop or CI to gate development iterations or CI pass/fail". - Replace the language-less arrow code-fence (rendered as a stray 'text' block on docs.page) with prose across README, index and why. - Remove the New Arch/TurboModule Info callout from the homepage (it stays in the guides). - Recapture the ios/ Codecov file table with charts hidden so the sunburst no longer appears twice.
The new pitch already frames coverage as a signal for your dev loop or CI, so the adjacent CTA caption was saying the same thing twice. Keep the Codecov dashboard image; drop the duplicate line.
Keep the hero dashboard image (the sunburst overview) and the line-by-line Coverage.mm shot; remove the ios/ file-table screenshot that duplicated the dashboard's tree. The ios/ Codecov link is folded into the surrounding sentence so nothing is lost.
mikehardy
force-pushed
the
docs/homepage-refinements
branch
from
September 24, 2026 20:23
0a27915 to
8e329ec
Compare
Collaborator
Author
|
🎉 This PR is included in version 0.2.2 🎉 The release is available on: Your semantic-release bot 📦🚀 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Follow-on to #2, addressing review feedback on the rendered homepage at docs.page/invertase/react-native-coverage.
What changed
Hero / heading
react-native-coverage). Broadened from "Native code coverage" to "Code coverage" since we also handle the TypeScript/JS side, and addedTypescriptto the platform list. Italics dropped fromiOS and Android.descriptionas the subtitle, so the bold body line was a repeat).Pitch flow (install → what it does)
.profrawand the app binary into clean LCOV / Jacoco, and remap instrumented JS line-for-line back to TypeScript — "smoothing the rough edges of modern TypeScript development by handling the line-accurate source mapping for you."The stray "text" block
Coverage.flush() → rn-coverage pull → …) rendered on docs.page as a bare code block with atextlanguage pill + copy button. Replaced with prose acrossREADME.md,docs/index.mdx, anddocs/why.mdx.Homepage Info callout
Infocallout from the homepage. It still lives in the guides where the prerequisite matters.Duplicate sunburst
ios/Codecov file table with charts hidden, so the sunburst pie no longer appears in both the hero dashboard image and the show-don't-tell shot.Verification
npx @docs.page/cli check→ No documentation issues found.