Merge branch 'production' into generate-description-util

Author: kodster28

.github/CODEOWNERS

@@ -92,6 +92,7 @@
 # Developer Platform
 
 /src/content/docs/containers/ @mikenomitch @th0m @cloudflare/pcx-technical-writing @cloudflare/cloudchamber
+/src/content/partials/containers/ @mikenomitch @th0m @cloudflare/pcx-technical-writing @cloudflare/cloudchamber
 /src/content/docs/d1/ @elithrar @rozenmd @vy-ton @joshthoward @oxyjun @harshil1712 @cloudflare/pcx-technical-writing
 /src/content/release-notes/d1.yaml @elithrar @rozenmd @vy-ton @joshthoward @oxyjun @cloudflare/pcx-technical-writing
 /src/content/partials/d1/ @elithrar @rozenmd @vy-ton @joshthoward @oxyjun @harshil1712 @cloudflare/pcx-technical-writing
@@ -103,7 +104,7 @@
 /src/content/release-notes/hyperdrive.yaml @elithrar @thomasgauvin @sejoker @oxyjun @cloudflare/pcx-technical-writing
 /src/content/partials/hyperdrive/ @elithrar @thomasgauvin @sejoker @oxyjun @cloudflare/pcx-technical-writing
 /src/content/release-notes/durable-objects.yaml @elithrar @vy-ton @joshthoward @oxyjun @cloudflare/pcx-technical-writing
-/src/content/docs/images/ @ToriLindsay @cloudflare/pcx-technical-writing @renandincer @third774
+/src/content/docs/images/ @ToriLindsay @cloudflare/pcx-technical-writing @renandincer @third774 @deanna-cf
 /src/content/docs/pages/ @cloudflare/workers-docs @GregBrimble @WalshyDev @aninibread @irvinebroque @cloudflare/pcx-technical-writing @kodster28
 /src/assets/images/pages/ @cloudflare/workers-docs @GregBrimble @WalshyDev @aninibread @cloudflare/pcx-technical-writing @kodster28
 /src/content/release-notes/pages.yaml @cloudflare/workers-docs @GregBrimble @WalshyDev @aninibread @cloudflare/pcx-technical-writing

.github/pull_request_template.md

@@ -11,6 +11,6 @@
 <!-- Remove items that do not apply -->
 
 - [ ] Is there a [changelog](https://developers.cloudflare.com/changelog/) entry ([guidelines](https://developers.cloudflare.com/style-guide/documentation-content-strategy/content-types/changelog/))? If you don't add one for something awesome and new (however small) — how will our customers find out? Changelogs are automatically posted to [RSS feeds](https://developers.cloudflare.com/fundamentals/new-features/available-rss-feeds/), the [Discord](https://discord.com/channels/595317990191398933/1040420029080018945), and [X](https://x.com/CFchangelog).
-- [ ] The [documentation style guide](https://developers.cloudflare.com/style-guide/) has been adhered to.
+- [ ] The change adheres to the [documentation style guide](https://developers.cloudflare.com/style-guide/).
 - [ ] If a larger change - such as adding a new page- an issue has been opened in relation to any incorrect or out of date information that this PR fixes.
 - [ ] Files which have changed name or location have been allocated [redirects](https://developers.cloudflare.com/pages/configuration/redirects/#per-file).

bin/validate-redirects.ts

@@ -6,6 +6,9 @@ async function main() {
 	let numInfiniteRedirects = 0;
 	let numUrlsWithFragment = 0;
 	let numDuplicateRedirects = 0;
+	let numNonSlashedRedirects = 0;
+
+	const validEndings = ["/", "*", ".xml", ".md", ".json", ".html"];
 
 	const redirectSourceUrls: string[] = [];
 
@@ -24,6 +27,11 @@ async function main() {
 			numUrlsWithFragment++;
 		}
 
+		if (!validEndings.some((ending) => from.endsWith(ending))) {
+			console.log(`✘ Found unslashed source URLs:\n    ${from}`);
+			numNonSlashedRedirects++;
+		}
+
 		if (redirectSourceUrls.includes(from)) {
 			console.log(`✘ Found repeated source URL:\n    ${from}`);
 			numDuplicateRedirects++;
@@ -32,7 +40,12 @@ async function main() {
 		}
 	}
 
-	if (numInfiniteRedirects || numUrlsWithFragment || numDuplicateRedirects) {
+	if (
+		numInfiniteRedirects ||
+		numUrlsWithFragment ||
+		numDuplicateRedirects ||
+		numNonSlashedRedirects
+	) {
 		console.log("\nDetected errors:");
 
 		if (numInfiniteRedirects > 0) {
@@ -47,6 +60,12 @@ async function main() {
 			console.log(`- ${numDuplicateRedirects} repeated source URL(s)`);
 		}
 
+		if (numNonSlashedRedirects > 0) {
+			console.log(
+				`- ${numNonSlashedRedirects} need slashes at the end of the source URL`,
+			);
+		}
+
 		console.log("\nPlease fix the errors above before merging :)");
 		process.exit(1);
 	} else {