🔗 Remove trailing slashes from Markdown links

[JFWooten4]

Re: #2675 (review) (point 1) and #2672 (review)

There were a few commits after #2409 since it was three months ago. That's why I tried to merge into main in #2583.

Styles are all aligned now.

[Copilot]

Pull request overview

Normalizes external Markdown links by removing trailing slashes, consolidating cleanup previously excluded from unrelated PRs.

Changes:

• Standardizes external URLs across validator, tooling, API, analytics, and build documentation.
• Updates the contributor guide’s pnpm link.
• Leaves link destinations and referral parameters otherwise unchanged.

Recommendation: MERGE-READY — the cleanup is complete and mechanically consistent.

Reviewed changes

Copilot reviewed 10 out of 10 changed files in this pull request and generated no comments.

Show a summary per file

File	Description
docs/validators/tier-1-orgs.mdx	Normalizes Obsrvr Radar links.
docs/validators/admin-guide/prerequisites.mdx	Normalizes the chrony link.
docs/validators/admin-guide/monitoring.mdx	Normalizes the Radar link.
docs/validators/admin-guide/configuring.mdx	Normalizes the Radar link.
docs/tools/developer-tools/wallets.mdx	Normalizes wallet-provider links.
docs/data/apis/rpc/providers.mdx	Normalizes the GetBlock link.
docs/data/analytics/analytics-providers/analytics-providers.mdx	Normalizes the commented Allium link.
docs/build/building-with-ai.mdx	Normalizes Stellar Skills links.
docs/build/apps/privacy.mdx	Normalizes privacy-demo links.
CONTRIBUTING.md	Normalizes the pnpm link.

[ElliotFriend]

Thanks for splitting these out, John, and sorry for the mixed signal.

My #2675 wording ("the standalone sweep, bundled into unrelated work") was ambiguous, you reasonably read the objection as the bundling, and Bri merging #2409 the week before muddied it further. That's on me.

But it sends us back to the question I asked on #2583 and never got an answer to: is there a source for the no-trailing-slash "rule," or is it more a personal preference? trailingSlash: false controls how Docusaurus generates output URLs, not how we author links in source, and #2409 had no description. I don't think we ever actually decided this.

So either:

• it's a convention, in which case it belongs in CONTRIBUTING.md with a lint rule so nobody has to hand-edit or review them again, or
• it isn't, and we leave existing links alone.

Both answers make this PR unnecessary, so I'm closing it as parked rather than reviewing it. Not a rejection.

Do you know where the rule originated? You're closer to this thread than I am, and if there's a real reason (SEO, redirects, something in the build) I'd rather hear it before deciding.