Skip to content

Commit 0fa21e8

Browse files
committed
Merge branch 'copilot/fix-style-issues-in-docs' of https://github.com/doc-detective/doc-detective.github.io into copilot/fix-style-issues-in-docs
2 parents b3fae9e + 2d289c3 commit 0fa21e8

27 files changed

+68
-60
lines changed

.vale.ini

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -18,3 +18,7 @@ Google.Headings = warning
1818
# Google.Quotes requires American punctuation style which can conflict with
1919
# code examples and technical terms - relax to warning
2020
Google.Quotes = warning
21+
22+
# Vale.Spelling doesn't respect vocabulary in frontmatter/titles
23+
# Relax to warning for technical docs with API terms
24+
Vale.Spelling = warning

docs/contribute/docs/local-development.mdx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -106,7 +106,7 @@ Use VS Code Dev Containers for a zero-config environment:
106106
npm run start
107107
```
108108

109-
Everything is pre-configured and ready to go!
109+
Everything is pre-configured and ready to go.
110110

111111
### Option 2: Local installation
112112

docs/contribute/docs/quick-start.mdx

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -28,15 +28,15 @@ Every documentation page has an **Edit this page** link at the bottom. This is t
2828
4. __Preview your changes__: Open a preview panel (Ctrl+Shift+V) to see how it looks.
2929
5. __Commit your changes__:
3030
1. In the side panel, click the **Source Control** button (looks like a branch icon).
31-
1. Add a brief description of what you changed (e.g., "Fix typo in installation steps").
31+
1. Add a brief description of what you changed (for example, "Fix typo in installation steps").
3232
1. Click "Commit & Push".
3333
6. __Create a pull request__:
3434
1. Review the changes one more time.
3535
1. In the Source Control panel, click the **Create Pull Request** button (looks like a vertical line with dots on both ends with an arrow pointing to the top and a plus sign).
3636
1. Enter a name and description for the pull request.
3737
1. Click **Create** to submit.
3838

39-
That's it! A maintainer will review your contribution.
39+
That's it. A maintainer will review your contribution.
4040

4141
## Tips for quick contributions
4242

@@ -70,7 +70,7 @@ For different issues across different pages, create separate pull requests—it'
7070
- Build verification
7171
2. __Review__: A maintainer reviews your changes (usually within 2 business days)
7272
3. __Feedback__: If changes are needed, the reviewer will comment on your pull request
73-
4. __Merge__: Once approved, your contribution is merged and goes live!
73+
4. __Merge__: Once approved, your contribution is merged and goes live.
7474

7575
## Common contributor concerns (and why you don't need to worry)
7676

@@ -80,7 +80,7 @@ For quick fixes like typos and link updates, no testing is required. The automat
8080

8181
### What if the automated checks fail?
8282

83-
Don't worry! The reviewer will help you fix any issues. Most common failures are easily correctable:
83+
Don't worry. The reviewer will help you fix any issues. Most common failures are correctable:
8484
- __Vale failures__: Usually suggest alternative wording (often just suggestions)
8585
- __Link check failures__: Means a link is broken or inaccessible
8686
- __Build failures__: Usually means there's a syntax error in Markdown

docs/contribute/docs/review-process.mdx

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -50,7 +50,7 @@ In your pull request, you'll see status checks:
5050

5151
### If checks fail
5252

53-
Don't panic! Failed checks are common and usually easy to fix.
53+
Don't panic. Failed checks are common and can usually be fixed.
5454

5555
#### Vale failures
5656

@@ -279,7 +279,7 @@ Most disagreements resolve through discussion.
279279

280280
### When your PR is approved
281281

282-
Congratulations! Your contribution will be merged soon.
282+
Congratulations. Your contribution will be merged soon.
283283

284284
What happens next:
285285

@@ -289,11 +289,11 @@ What happens next:
289289

290290
### After merge
291291

292-
Celebrate! 🎉 You've contributed to open source! Post about it, add to your portfolio, tell your friends.
292+
Celebrate. 🎉 You've contributed to open source. Post about it, add to your portfolio, tell your friends.
293293

294294
If your PR addresses an issue, it should close automatically.
295295

296-
Found something else to improve? Open another PR!
296+
Found something else to improve? Open another PR.
297297

298298
## Special cases
299299

@@ -369,7 +369,7 @@ Life happens. If you can't address feedback right away,
369369
- **Suggest improvements**: Open issues for documentation gaps you notice
370370
- **Join the community**: Engage in Discord discussions
371371

372-
### You're now a contributor!
372+
### You're now a contributor
373373

374374
Thank you for making Doc Detective better. Your contribution, no matter how small, helps all of our users.
375375

docs/contribute/docs/standard-contributions.mdx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -245,7 +245,7 @@ Before creating a pull request:
245245
2. **Maintainer reviews** your contribution (within 5 business days)
246246
3. **Feedback or approval**:
247247
- If changes are needed, update your pull request.
248-
- If approved, your contribution is merged!
248+
- If approved, your contribution is merged.
249249

250250
### Understanding automated checks
251251

docs/contribute/docs/templates/feature.mdx

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -219,7 +219,7 @@ Examples should be copy-pasteable and work as-is:
219219
}
220220
```
221221

222-
### Document breaking changes clearly
222+
### Document breaking changes
223223

224224
Be explicit about what breaks:
225225

@@ -373,7 +373,7 @@ None. This is a backward-compatible addition. Existing screenshots continue to w
373373

374374
## Checklist before submitting
375375

376-
- [ ] Clearly explains what the feature is and why it matters
376+
- [ ] Explains what the feature is and why it matters
377377
- [ ] Provides complete, working examples
378378
- [ ] Documents all configuration options
379379
- [ ] Includes migration guide if replacing existing functionality

docs/contribute/docs/templates/reference.mdx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -164,7 +164,7 @@ Reference docs should cover unusual behavior:
164164

165165
✅ Complete:
166166

167-
> If `timeout` is set to `0`, the action waits indefinitely. If `timeout` is negative, it defaults to 5000ms.
167+
> If `timeout` is set to `0`, the action waits indefinitely. If `timeout` is negative, it defaults to 5000 ms.
168168
169169
⛔ Incomplete:
170170

docs/contribute/docs/templates/troubleshooting.mdx

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -181,7 +181,7 @@ Help users understand, don't just give commands:
181181
182182
✅ Educational:
183183
184-
> Increase the timeout to 10000ms (10 seconds) to give the page more time to load. CI environments are often slower than local machines.
184+
> Increase the timeout to 10000 ms (10 seconds) to give the page more time to load. CI environments are often slower than local machines.
185185
186186
⛔ Just commands:
187187
@@ -368,7 +368,7 @@ If you've tried these solutions and still have issues:
368368
369369
## Checklist before submitting
370370
371-
- [ ] Problem is clearly stated in the title
371+
- [ ] Problem is stated in the title
372372
- [ ] Symptoms are specific and observable
373373
- [ ] Exact error messages are included
374374
- [ ] Root cause is explained

docs/contribute/docs/templates/tutorial.mdx

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -56,7 +56,7 @@ description: [What users will learn and accomplish]
5656

5757
- Prerequisite knowledge or skill
5858
- Required tool or software
59-
- Estimated time (e.g., "30 minutes")
59+
- Estimated time (for example, "30 minutes")
6060

6161
## Step 1: [First major concept or action]
6262

@@ -148,14 +148,14 @@ Each step should build on previous ones:
148148

149149
✅ Good progression:
150150

151-
1. Create a simple test with one action
151+
1. Create a basic test with one action
152152
2. Add a second action to create a flow
153153
3. Add error handling
154154
4. Organize into a test suite
155155

156156
⛔ Poor progression:
157157

158-
1. Create a simple test
158+
1. Create a basic test
159159
2. Deploy to production (too big a jump)
160160

161161
### Explain the why
@@ -176,7 +176,7 @@ Tutorials should be encouraging:
176176

177177
✅ Good:
178178

179-
> Great! You've created your first test. Let's make it more useful by adding another action.
179+
> Great. You've created your first test. Let's make it more useful by adding another action.
180180
181181
⛔ Too formal:
182182

@@ -194,7 +194,7 @@ After each major step, show what users should see:
194194
> [PASS] test-1: Passed (2.3s)
195195
> ```
196196
>
197-
> This confirms your test ran successfully!
197+
> This confirms your test ran successfully.
198198
199199
⛔ Missing:
200200
@@ -431,7 +431,7 @@ Now that you understand the basics:
431431
432432
## Checklist before submitting
433433
434-
- [ ] Clearly states what users will learn
434+
- [ ] States what users will learn
435435
- [ ] Lists what users will build
436436
- [ ] Prerequisites and time estimate included
437437
- [ ] Steps build progressively

docs/contribute/docs/testing.mdx

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -407,7 +407,7 @@ npx doc-detective runTests --input your-test.spec.json --logLevel debug
407407

408408
## Test best practices
409409

410-
### Keep tests simple
410+
### Keep tests straightforward
411411

412412
**Do:**
413413
- Test one concept per test
@@ -424,7 +424,7 @@ npx doc-detective runTests --input your-test.spec.json --logLevel debug
424424
### Make tests maintainable
425425

426426
**Do:**
427-
- Use comments to explain non-obvious steps
427+
- Use comments to explain complex steps
428428
- Group related tests in the same file
429429
- Use descriptive action parameters
430430
- Clean up after tests (delete files, etc.)
@@ -513,7 +513,7 @@ Maintainers review tests as part of PR review. They can help with:
513513
- Optimizing test performance
514514
- Deciding what to test
515515

516-
Don't hesitate to ask for help—testing documentation is a unique skill!
516+
Don't hesitate to ask for help—testing documentation is a unique skill.
517517

518518
## Next steps
519519

0 commit comments

Comments
 (0)