Skip to content

Fix typos and grammar in six documentation pages - #1445

Merged
yubiuser merged 2 commits into
pi-hole:masterfrom
TypoScan3000:docs/fix-typos
Oct 25, 2026
Merged

yubiuser merged 2 commits into
pi-hole:masterfrom
TypoScan3000:docs/fix-typos

Conversation

@Avicennasis

Copy link
Copy Markdown
Contributor

Thank you for your contribution to the Pi-hole Community!

What does this PR aim to accomplish?:

Fixes ten small typos and grammar slips in the documentation.

How does this PR accomplish the above?:

  • docs/api/tls.md: "...(e.g., /etc/pihole) and, change the path" → "and change the path"
  • docs/guides/misc/tor/performance-issues.md: "Edit /etc/tor/torrc as root and, add" → "as root and add"
  • docs/regex/pi-hole.md: "will block not block abc but everything else" → "will not block abc but everything else" (the ;invert example); "it used solely for readability" → "it is used solely"
  • docs/database/domain-database/index.md: "through the various view" → "views"; "will match al clients" → "match all clients"
  • docs/routers/OPNsense.md: "using a OPNsense 25.1.1" → "an OPNsense"; "block all DNS request that are" → "DNS requests that are"; "an Alias for Pi-holes MAC address" → "Pi-hole's MAC address"
  • docs/docker/tips-and-tricks.md: "can lead to situation where" → "to a situation where"

Testing: text-only substitutions inside existing lines (no line added or removed, LF endings kept). git diff shows only the words listed above. npm test was not run locally. MD013 is disabled in .markdownlint.json, and no Markdown structure changed. docs/ftldns/configfile.md is untouched.

Link documentation PRs if any are needed to support this PR:

N/A (this is the documentation PR). The single commit is signed off (DCO) and GPG-signed.


By submitting this pull request, I confirm the following:

  1. I have read and understood the contributors guide, as well as this entire template. I understand which branch to base my commits and Pull Requests against.
  2. I have commented my proposed changes within the code and I have tested my changes.
  3. I am willing to help maintain this change if there are issues with it later.
  4. It is compatible with the EUPL 1.2 license
  5. I have squashed any insignificant commits. (git rebase)
  6. I have checked that another pull request for this purpose does not exist.
  7. I have considered, and confirmed that this submission will be valuable to others.
  8. I accept that this submission may not be used, and the pull request closed at the will of the maintainer.
  9. I give this submission freely, and claim no ownership to its content.

  • I have read the above and my PR is ready for review. Check this box to confirm

No functional changes — comments/documentation only.

@Avicennasis
Avicennasis requested a review from a team as a code owner October 25, 2026 04:26
@netlify

netlify Bot commented Oct 25, 2026 •

Copy link
Copy Markdown

✅ Deploy Preview for pihole-docs ready!

Name Link
🔨 Latest commit 8313034
🔍 Latest deploy log https://app.netlify.com/projects/pihole-docs/deploys/6adda0172a952d89f68d2d3b
😎 Deploy Preview https://deploy-preview-1445--pihole-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
🤖 Make changes Run an agent on this branch

To edit notification comments on pull requests, go to your Netlify project configuration.

Comment thread docs/routers/OPNsense.md Outdated
Avicennasis and others added 2 commits October 25, 2026 07:22
- api/tls.md, guides/misc/tor/performance-issues.md: drop stray comma in "and, change" / "and, add"
- regex/pi-hole.md: "will block **not** block" -> "will **not** block"; "it used solely" -> "it is used solely"
- database/domain-database/index.md: "various view" -> "various views"; "match al clients" -> "match all clients"
- routers/OPNsense.md: "a OPNsense" -> "an OPNsense"; "DNS request that are" -> "DNS requests that are"; "Pi-holes MAC" -> "Pi-hole's MAC"
- docker/tips-and-tricks.md: "lead to situation" -> "lead to a situation"

Signed-off-by: Léon Avic Simmons <Avicennasis@gmail.com>
Signed-off-by: yubiuser <github@yubiuser.dev>
@yubiuser
yubiuser enabled auto-merge October 25, 2026 06:22
@yubiuser
yubiuser merged commit 0cb5bc3 into pi-hole:master Oct 25, 2026
13 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.

2 participants