Skip to content

refactor: replace route format with pattern in [website] redirect service#11858

Merged
jNullj merged 1 commit into
badges:masterfrom
jNullj:refactor-format-into-pattern/website-redirect
May 14, 2026
Merged

refactor: replace route format with pattern in [website] redirect service#11858
jNullj merged 1 commit into
badges:masterfrom
jNullj:refactor-format-into-pattern/website-redirect

Conversation

@jNullj
Copy link
Copy Markdown
Member

@jNullj jNullj commented May 13, 2026

replace route's deprecated format with pattern

part of #3329
helps #11800

@jNullj jNullj added the service-badge New or updated service badge label May 13, 2026
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 13, 2026

Warnings
⚠️ This PR modified service code for website but not its test code.
That's okay so long as it's refactoring existing code.
Messages
📖 ✨ Thanks for your contribution to Shields, @jNullj!

Generated by 🚫 dangerJS against 8ac17f4

replace `route`'s deprecated `format` with `pattern`

part of badges#3329
helps badges#11800
@jNullj jNullj force-pushed the refactor-format-into-pattern/website-redirect branch from 3fd7e88 to 8ac17f4 Compare May 13, 2026 22:06
@jNullj jNullj marked this pull request as ready for review May 13, 2026 22:11
Copy link
Copy Markdown
Member

@PyvesB PyvesB left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks fine to me. We are making the path a bit looser (e.g. /website-foo/https/host.svg would previously 404 and now doesn't), but given this is a deprecated redirector, I don't think we should invest too much time in polishing it or making it bullet-proof.

@jNullj jNullj added this pull request to the merge queue May 14, 2026
Merged via the queue into badges:master with commit 46caa19 May 14, 2026
19 checks passed
@jNullj jNullj deleted the refactor-format-into-pattern/website-redirect branch May 14, 2026 09:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

service-badge New or updated service badge

Development

Successfully merging this pull request may close these issues.

3 participants