Skip to content

caddyhttp: dedupe MatchPath godoc for %* wildcards - #7932

Open
locker95 wants to merge 1 commit into
caddyserver:masterfrom
locker95:docs/path-matcher-dedupe
Open

caddyhttp: dedupe MatchPath godoc for %* wildcards#7932
locker95 wants to merge 1 commit into
caddyserver:masterfrom
locker95:docs/path-matcher-dedupe

Conversation

@locker95

@locker95 locker95 commented Aug 8, 2026

Copy link
Copy Markdown

The path matcher docs on the website (and the godoc that feeds them) had two near-identical paragraphs about %*. Folded them into one that still shows both examples.

Fixes caddyserver/website#484

Assistance Disclosure

No AI tooling on this change — small godoc edit by hand.

@CLAassistant

Copy link
Copy Markdown

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

@steadytao

Copy link
Copy Markdown
Member

Could you sign the CLA and add our required assistance disclosure?

@locker95

Copy link
Copy Markdown
Author

Added the assistance disclosure on the PR. I'll get the CLA signed if it's still pending — think I signed before but happy to re-do it.

@francislavoie

Copy link
Copy Markdown
Member

Make sure the email you used to commit with matches one of the emails on your github account. CLA is still not signed.

The JSON docs surface this comment twice with nearly the same wording.
Keep one paragraph that covers both examples.

Signed-off-by: Dean Chen <862469039@qq.com>
@locker95

Copy link
Copy Markdown
Author

Good catch — commit author was on a qq.com address. Rewrote it to my GitHub noreply email and force-pushed. Signing the CLA with that next if the badge still lags.

@locker95
locker95 force-pushed the docs/path-matcher-dedupe branch from 12e7ca9 to bce87c6 Compare August 10, 2026 08:44
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.

Duplicate paragraphs on /docs/json/apps/http/servers/routes/match/path/

4 participants