Skip to content

[Chore] Remove outdated file#30901

Merged
kodster28 merged 3 commits into
productionfrom
video-index-cleanup
May 19, 2026
Merged

[Chore] Remove outdated file#30901
kodster28 merged 3 commits into
productionfrom
video-index-cleanup

Conversation

@kodster28
Copy link
Copy Markdown
Collaborator

@kodster28 kodster28 commented May 18, 2026

Summary

Remove outdated src/content/videos/index.yaml file. This hasn't been actively maintained and -- as such -- isn't the best reference for the information we have available on Cloudflare.

@github-actions github-actions Bot added product:queues Cloudflare Queues: https://developers.cloudflare.com/queues product:d1 D1: https://developers.cloudflare.com/d1/ product:style-guide product:kv product:ai-gateway AI Gateway: https://developers.cloudflare.com/ai-gateway/ product:workers-ai Workers AI: https://developers.cloudflare.com/workers-ai/ product:vectorize Vectorize: https://developers.cloudflare.com/vectorize/ size/l labels May 18, 2026
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 18, 2026

This PR requires additional review attention because it affects the following areas:

Redirects

This PR changes current filenames or deletes current files. Make sure you have redirects set up to cover the following paths:

  • /style-guide/components/youtube-videos/

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 18, 2026 •
edited
Loading

This pull request requires reviews from CODEOWNERS as it changes files that match the following patterns:

Pattern Owners
* @cloudflare/product-owners
*.ts @cloudflare/content-engineering, @kodster28
*.astro @cloudflare/content-engineering, @kodster28
/src/content/docs/ai-gateway/ @abhishekkankani, @palashgo, @thebongy, @roerohan, @kathayl, @mchenco, @zeke, @superhighfives, @bfirsh, @mattrothenberg, @ethulia, @cloudflare/product-owners
/src/content/docs/cloudflare-one/ @ranbel, @Maddy-Cloudflare, @codyanthony850, @cloudflare/cf1-reviewers, @elithrar, @cloudflare/product-owners
/src/content/docs/d1/ @elithrar, @rozenmd, @vy-ton, @joshthoward, @oxyjun, @harshil1712, @cloudflare/product-owners
/src/content/docs/kv/ @elithrar, @thomasgauvin, @oxyjun, @cloudflare/product-owners
/src/content/docs/pages/ @cloudflare/workers-docs, @GregBrimble, @WalshyDev, @aninibread, @irvinebroque, @cloudflare/product-owners, @MattieTK
/src/content/docs/queues/ @elithrar, @jonesphillip, @harshil1712, @mia303, @cloudflare/product-owners
/src/content/docs/r2/ @oxyjun, @elithrar, @jonesphillip, @aninibread, @harshil1712, @cloudflare/workers-docs, @cloudflare/product-owners
/src/content/docs/style-guide/ @dcpena, @caley-b, @cloudflare/product-owners
/src/content/docs/vectorize/ @elithrar, @vy-ton, @sejoker, @mchenco, @cloudflare/product-owners
/src/content/docs/workers-ai/ @rita3ko, @craigsdennis, @mchenco, @zeke, @superhighfives, @bfirsh, @mattrothenberg, @ethulia, @cloudflare/product-owners
/src/content/docs/workers/ @cloudflare/workers-docs, @GregBrimble, @irvinebroque, @mikenomitch, @korinne, @WalshyDev, @cloudflare/deploy-config, @cloudflare/product-owners, @cloudflare/wrangler, @MattieTK, @cloudflare/dev-plat-leads

ask-bonk[bot]
ask-bonk Bot reviewed May 18, 2026
View reviewed changes
Copy link
Copy Markdown
Contributor

@ask-bonk ask-bonk Bot left a comment

Choose a reason for hiding this comment

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

Type-check and lint pass. One issue needs fixing before merge.

MEDIUM — Missing redirect for deleted page. Add to :

@ask-bonk
Copy link
Copy Markdown
Contributor

ask-bonk Bot commented May 18, 2026

Missing redirect line to add to public/__redirects:

/style-guide/components/youtube-videos/ /style-guide/components/youtube/ 301

@ask-bonk
Copy link
Copy Markdown
Contributor

ask-bonk Bot commented May 18, 2026

I reviewed PR #30901. Here's the summary:

What the PR changes:
Removes the outdated src/content/videos/index.yaml collection, the YouTubeVideos Astro component, its schema, and all references across tutorial index pages and agent reference files. The ExternalResources component is updated to only support type="apps" (no longer "videos").

Validation results:

  • pnpm run check — passed (0 errors)
  • pnpm run lint — passed
  • pnpm run format:core:check — passed

Issue flagged:
MEDIUM — Missing redirect for the deleted style-guide page. I posted a review comment on PR #30901 requesting that the following line be added to public/__redirects:

/style-guide/components/youtube-videos/ /style-guide/components/youtube/ 301

Labels: The PR already has appropriate product labels (product:workers, product:pages, product:r2, product:queues, product:d1, product:style-guide, product:kv, product:ai-gateway, product:workers-ai, product:vectorize, size/l). No additional labels were necessary.

github run

ahaywood
ahaywood reviewed May 18, 2026
View reviewed changes
Copy link
Copy Markdown
Contributor

@ahaywood ahaywood left a comment

Choose a reason for hiding this comment

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

Approved with the changes Bonk requested.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 18, 2026 •
edited
Loading

Preview URL: https://c7dbd956.preview.developers.cloudflare.com
Preview Branch URL: https://video-index-cleanup.preview.developers.cloudflare.com

Files with changes (up to 15)

Original Link Updated Link
https://developers.cloudflare.com/style-guide/components/youtube-videos/ https://video-index-cleanup.preview.developers.cloudflare.com/style-guide/components/youtube-videos/
https://developers.cloudflare.com/r2/tutorials/ https://video-index-cleanup.preview.developers.cloudflare.com/r2/tutorials/
https://developers.cloudflare.com/style-guide/components/external-resources/ https://video-index-cleanup.preview.developers.cloudflare.com/style-guide/components/external-resources/
https://developers.cloudflare.com/workers-ai/guides/tutorials/ https://video-index-cleanup.preview.developers.cloudflare.com/workers-ai/guides/tutorials/
https://developers.cloudflare.com/pages/tutorials/ https://video-index-cleanup.preview.developers.cloudflare.com/pages/tutorials/
https://developers.cloudflare.com/queues/tutorials/ https://video-index-cleanup.preview.developers.cloudflare.com/queues/tutorials/
https://developers.cloudflare.com/ai-gateway/tutorials/ https://video-index-cleanup.preview.developers.cloudflare.com/ai-gateway/tutorials/
https://developers.cloudflare.com/d1/tutorials/ https://video-index-cleanup.preview.developers.cloudflare.com/d1/tutorials/
https://developers.cloudflare.com/kv/tutorials/ https://video-index-cleanup.preview.developers.cloudflare.com/kv/tutorials/
https://developers.cloudflare.com/vectorize/tutorials/ https://video-index-cleanup.preview.developers.cloudflare.com/vectorize/tutorials/
https://developers.cloudflare.com/workers/tutorials/ https://video-index-cleanup.preview.developers.cloudflare.com/workers/tutorials/
https://developers.cloudflare.com/cloudflare-one/video-tutorials/ https://video-index-cleanup.preview.developers.cloudflare.com/cloudflare-one/video-tutorials/

@kodster28 kodster28 merged commit cd6721d into production May 19, 2026
11 checks passed
@kodster28 kodster28 deleted the video-index-cleanup branch May 19, 2026 15:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@ahaywood ahaywood ahaywood approved these changes

@ask-bonk ask-bonk[bot] ask-bonk[bot] left review comments

@GregBrimble GregBrimble Awaiting requested review from GregBrimble GregBrimble is a code owner

@irvinebroque irvinebroque Awaiting requested review from irvinebroque irvinebroque is a code owner

@mikenomitch mikenomitch Awaiting requested review from mikenomitch mikenomitch is a code owner

@korinne korinne Awaiting requested review from korinne korinne is a code owner

@WalshyDev WalshyDev Awaiting requested review from WalshyDev WalshyDev is a code owner

@MattieTK MattieTK Awaiting requested review from MattieTK MattieTK is a code owner

@Oxyjun Oxyjun Awaiting requested review from Oxyjun Oxyjun is a code owner

@elithrar elithrar Awaiting requested review from elithrar elithrar is a code owner

@jonesphillip jonesphillip Awaiting requested review from jonesphillip jonesphillip is a code owner

@aninibread aninibread Awaiting requested review from aninibread aninibread is a code owner

@harshil1712 harshil1712 Awaiting requested review from harshil1712 harshil1712 is a code owner

@mia303 mia303 Awaiting requested review from mia303 mia303 is a code owner

@thomasgauvin thomasgauvin Awaiting requested review from thomasgauvin thomasgauvin is a code owner

@rozenmd rozenmd Awaiting requested review from rozenmd rozenmd is a code owner

@vy-ton vy-ton Awaiting requested review from vy-ton vy-ton is a code owner

@joshthoward joshthoward Awaiting requested review from joshthoward joshthoward is a code owner

@sejoker sejoker Awaiting requested review from sejoker sejoker is a code owner

@mchenco mchenco Awaiting requested review from mchenco mchenco is a code owner

@rita3ko rita3ko Awaiting requested review from rita3ko rita3ko is a code owner

@craigsdennis craigsdennis Awaiting requested review from craigsdennis craigsdennis is a code owner

@zeke zeke Awaiting requested review from zeke zeke is a code owner

@superhighfives superhighfives Awaiting requested review from superhighfives superhighfives is a code owner

@mattrothenberg mattrothenberg Awaiting requested review from mattrothenberg mattrothenberg is a code owner

@ethulia ethulia Awaiting requested review from ethulia ethulia is a code owner

@palashgo palashgo Awaiting requested review from palashgo palashgo is a code owner

@roerohan roerohan Awaiting requested review from roerohan roerohan is a code owner

@kathayl kathayl Awaiting requested review from kathayl kathayl is a code owner

Assignees

@zeke zeke

@elithrar elithrar

@superhighfives superhighfives

@mattrothenberg mattrothenberg

@ethulia ethulia

@thebongy thebongy

@mchenco mchenco

@palashgo palashgo

@roerohan roerohan

@kathayl kathayl

Labels

product:ai-gateway AI Gateway: https://developers.cloudflare.com/ai-gateway/ product:d1 D1: https://developers.cloudflare.com/d1/ product:kv product:pages product:queues Cloudflare Queues: https://developers.cloudflare.com/queues product:r2 R2 object storage: https://developers.cloudflare.com/r2 product:style-guide product:vectorize Vectorize: https://developers.cloudflare.com/vectorize/ product:workers Related to Workers product product:workers-ai Workers AI: https://developers.cloudflare.com/workers-ai/ size/l

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.