📝 Remove dead doc hub pages#6833
Conversation
These 12 index.md pages contained nothing but a title and a <DocCardList /> component, adding an unnecessary click to the user journey. Replace them with _category_.json files to preserve sidebar ordering, and update all internal links to point directly to meaningful content pages. https://claude.ai/code/session_01MUhihYg2fwBYzmLej4chb6
|
👋 It looks like the PR template was not used for this pull request. Please edit the PR description to use the template, or close this PR and open a new one that uses it. The template includes:
Thank you for your contribution! 🙏 |
|
@fast-check/ava
fast-check
@fast-check/jest
@fast-check/packaged
@fast-check/poisoning
@fast-check/vitest
@fast-check/worker
commit: |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #6833 +/- ##
=======================================
Coverage 94.86% 94.86%
=======================================
Files 209 209
Lines 5665 5665
Branches 1488 1488
=======================================
Hits 5374 5374
Misses 283 283
Partials 8 8
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
| { to: '/docs/tutorials/', label: 'All Tutorials' }, | ||
| { to: '/docs/introduction/what-is-property-based-testing/', label: 'Documentation' }, | ||
| { to: '/docs/tutorials/quick-start/basic-setup/', label: 'Quick Start' }, | ||
| { to: '/docs/tutorials/quick-start/basic-setup/', label: 'All Tutorials' }, |
There was a problem hiding this comment.
@claude add back the index for tutorials. Try to make it richer compared to what we had in the past
There was a problem hiding this comment.
Claude finished @dubzzz's task in 5m 32s —— View job
Task: Add back a richer tutorials index
- Explore tutorials directory and check old index content
- Check
docusaurus.config.tsto see tutorials links - Create richer
website/docs/tutorials/index.md - Remove
_category_.json(replaced by index.md front matter) - Update
docusaurus.config.tslinks (navbar + footer) - Manually verify every slug referenced in the new index
- Commit and push changes
What changed
Brought back website/docs/tutorials/index.md, but instead of the previous title + <DocCardList /> hub, it now contains:
- Why follow a tutorial? — a short rationale and a tip pointing newcomers to the Property-Based Testing primers.
- Learning paths — three curated entry points:
- 🚀 I'm completely new to fast-check → numbered walkthrough of the Quick Start pages.
- 🧰 I want to plug fast-check into my existing test runner → direct links to the Jest / Bun / Deno / Node.js guides.
- 🏎️ I want to hunt bugs that only show up under concurrency → pointer to the race conditions tutorial.
- What's inside? — a summary table of each tutorial, what you learn, and who it fits.
- Looking for something more specific? — cross-links to Core Blocks, Configuration, Advanced and the API reference so the index doubles as a launchpad into the rest of the docs.
- A closing feedback welcome info box pointing to the GitHub issue tracker.
Also updated:
website/docusaurus.config.ts— navbar and footer All Tutorials entries now point to/docs/tutorials/again (they had been rerouted to the Quick Start page when the old hub was deleted).- Removed
website/docs/tutorials/_category_.json; the newindex.mdcarriessidebar_position: 5in its front matter, matching the pattern already used by the sibling tutorials (detect-race-conditions,setting-up-your-test-environment).
All slugs referenced in the new index were verified to exist in the repo. The project's Stop hook will run build:all / lint:check / typecheck:all on completion, which will additionally exercise Docusaurus' onBrokenLinks: 'throw' check.
Commit: 4463d14 on claude/remove-dead-doc-pages-V2qGc
· Branch
Bring back website/docs/tutorials/index.md (removed in this PR as a dead DocCardList hub) but make it actually useful: describe what a tutorial is in fast-check, surface learning paths tailored to a few common entry points, summarise each tutorial in a table, and link out to the rest of the docs and API reference for deeper reading. Point the "All Tutorials" navbar and footer entries back at the dedicated index instead of redirecting to the Quick Start. Co-authored-by: Nicolas DUBIEN <[email protected]>
|
|
||
| Pick the path that matches where you are today. | ||
|
|
||
| ### 🚀 I'm completely new to fast-check |
There was a problem hiding this comment.
@claude prefer putting the doccard compo for child pages rather than putting details manually for them
There was a problem hiding this comment.
Claude finished @dubzzz's task in 2m 54s —— View job
Task: Replace manual child-page details with DocCardList
- Read current
website/docs/tutorials/index.md - Check existing
DocCardListusage in the repo (setting-up-your-test-environment/index.mduses themdx-code-blockpattern) - Rewrite index to use
<DocCardList />for child pages, keeping the richer surrounding sections - Commit and push
What changed
website/docs/tutorials/index.md no longer duplicates each child page's description by hand. Specifically:
- Removed the
## Learning pathssubsections that manually enumerated individual Quick Start pages (Basic Setup, Our first Property-Based Test, Read test reports) and each Setting up your Test Environment runner (Jest, Bun, Deno, Node.js). - Removed the
## What's inside?markdown table that re-described each tutorial. - Added a concise
## Which tutorial is right for me?block — three one-line routing bullets by intent (new to fast-check / integrating a runner / chasing race conditions), each pointing at the corresponding tutorial landing page. - Added a
## All tutorialssection backed by<DocCardList />(imported via themdx-code-blockfence, matching the pattern already used insetting-up-your-test-environment/index.md). The cards are now driven by each child page's own front matter, so descriptions stay in a single source of truth. - Kept the
## Why follow a tutorial?intro, the PBT primers tip, the## Looking for something more specific?cross-links (Core Blocks, Configuration, Advanced, API Reference), and the closing feedback info box.
Net diff on this file: +10 / -27.
Commit: 04fa2e5 on claude/remove-dead-doc-pages-V2qGc
· Branch
Co-authored-by: Nicolas DUBIEN <[email protected]>
| - [**API Reference**](https://fast-check.dev/api-reference/index.html) — the exhaustive list of every exported symbol. | ||
|
|
||
| :::info Got feedback or a tutorial idea? | ||
| Tutorials improve when real users tell us what's missing. If a step is unclear, or you'd love to see a tutorial on a topic we don't cover yet, please [open an issue](https://github.com/dubzzz/fast-check/issues) or start a discussion on the repository. Contributions are very welcome! 💙 |
There was a problem hiding this comment.
@claude only link to discussions, limit issues to bug report. Probably not worth mentioning in this page
There was a problem hiding this comment.
Claude finished @dubzzz's task in 1m 17s —— View job
Task: Remove feedback info box from tutorials index
- Read current
website/docs/tutorials/index.md - Remove the feedback info box (not worth mentioning here)
- Commit and push
What changed
Dropped the :::info Got feedback or a tutorial idea? box from website/docs/tutorials/index.md. It previously pointed users at both the issue tracker and discussions for tutorial ideas, which conflicts with the policy of reserving issues for bug reports — and, as you noted, it's not really worth mentioning on this page at all. The rest of the index (intro, routing bullets, <DocCardList />, cross-links to Core Blocks / Configuration / Advanced / API Reference) is untouched.
Commit: 75b6c6d on claude/remove-dead-doc-pages-V2qGc
· Branch ·
Co-authored-by: Nicolas DUBIEN <[email protected]>
|
@claude we usually use relative links on our doc, make sure you follow what we usually do on all other pages |
|
Claude finished @dubzzz's task in 6m 56s —— View job Task: Use relative links on tutorials index
What changedConverted every internal link in
The external A note on the conventionI searched every Commit: |
Co-authored-by: Nicolas DUBIEN <[email protected]>
|
@claude not sur we need the .MD try to compile and build and check how we usually reference other pages. Fix buggy relative links if needed |
|
Claude finished @dubzzz's task in 5m 10s —— View job Task: Align tutorials index links to repo convention
What I foundI grepped every — i.e. an absolute slug-style path, with trailing slash (matching What changedRewrote every internal link in
The external API reference link ( I cross-checked every new URL against the
Net diff on the file: +7 / -7. A note on the local buildYou asked me to try to compile and build. I tried — Commit: |
Every other doc page in website/docs uses /docs/...-style slug links (with trailing slash, no .md extension). Switch the tutorials index to the same style so it stays consistent. Co-authored-by: Nicolas DUBIEN <[email protected]>
These 12 index.md pages contained nothing but a title and a
component, adding an unnecessary click to the
user journey. Replace them with category.json files to preserve
sidebar ordering, and update all internal links to point directly
to meaningful content pages.
https://claude.ai/code/session_01MUhihYg2fwBYzmLej4chb6