Skip to content

Conversation

caugner
Copy link
Contributor

@caugner caugner commented Sep 1, 2025

Description

Makes fred the default frontend for:

  • the yarn start script
  • the pr-test workflow (used by the pr-review-companion)

See also: https://github.com/mdn/fred/

Note: The existing start script is preserved as start:legacy for now, but will be removed soon.

Motivation

Ensures that PR authors, and reviewers have an experience that is consistent with the deployed MDN site.

Additional details

Related issues and pull requests

Fixes mdn/fred#667.

@caugner caugner requested a review from mdn-bot as a code owner September 1, 2025 10:08
@github-actions github-actions bot added system [PR only] Infrastructure and configuration for the project size/s [PR only] 6-50 LoC changed labels Sep 1, 2025
@caugner caugner requested a review from LeoMcA September 1, 2025 14:05
@caugner caugner marked this pull request as draft September 1, 2025 14:06
@caugner

This comment was marked as outdated.

Migrates the `yarn start` command to the new MDN front end (fred).

See also: https://github.com/mdn/fred/

The existing command is preserved with a `:legacyi` suffix for now.
@caugner caugner force-pushed the fred-667-migrate-to-fred branch from d2781da to 55f9f54 Compare September 1, 2025 17:01
@caugner caugner marked this pull request as ready for review September 1, 2025 17:05
@caugner caugner marked this pull request as draft September 1, 2025 17:08
@github-actions github-actions bot added Content:Learn Learning area docs Content:Other Any docs not covered by another "Content:" label Content:Games Games docs Content:Meta Content in the meta docs labels Sep 1, 2025
Copy link
Contributor

github-actions bot commented Sep 1, 2025

Preview URLs

Flaws (5)

Note! 2 documents with no flaws that don't need to be listed. 🎉

URL: /en-US/docs/Learn_web_development
Title: Learn web development
Flaw count: 3

  • unknown:
    • No generic content config found
    • no blog root
    • no blog root

URL: /en-US/docs/MDN
Title: MDN Web Docs
Flaw count: 1

  • broken_links:
    • Can't resolve /en-US/about

URL: /en-US/docs/Web
Title: Web technology for developers
Flaw count: 1

  • broken_links:
    • Can't resolve /en-US/docs/Web/LMTH
External URLs (3)

URL: /en-US/docs/Games
Title: Game development


URL: /en-US/docs/Web
Title: Web technology for developers

(comment last updated: 2025-09-03 16:10:31)

@caugner caugner force-pushed the fred-667-migrate-to-fred branch from 2d2714f to 784d15b Compare September 1, 2025 17:23
@github-actions github-actions bot added size/m [PR only] 51-500 LoC changed and removed size/s [PR only] 6-50 LoC changed labels Sep 3, 2025
@caugner caugner changed the title feat(scripts): migrate start command to new fred front end feat: migrate start command + pr-test to fred Sep 3, 2025
@github-actions github-actions bot added size/s [PR only] 6-50 LoC changed and removed Content:Learn Learning area docs Content:Other Any docs not covered by another "Content:" label Content:Games Games docs Content:Meta Content in the meta docs size/m [PR only] 51-500 LoC changed labels Sep 3, 2025
@caugner caugner marked this pull request as ready for review September 3, 2025 16:14
@caugner caugner requested a review from a team as a code owner September 3, 2025 16:14
@caugner
Copy link
Contributor Author

caugner commented Sep 3, 2025

@LeoMcA This is ready for review now! 🎉

@caugner caugner changed the title feat: migrate start command + pr-test to fred feat: use fred by default Sep 3, 2025
Copy link
Member

@LeoMcA LeoMcA left a comment

Choose a reason for hiding this comment

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

Good stuff! I'll let you merge in case there's any final approval required from content, or you want to launch with certain timing 🚀

@caugner caugner merged commit f8bbd68 into main Sep 4, 2025
19 checks passed
@caugner caugner deleted the fred-667-migrate-to-fred branch September 4, 2025 07:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
size/s [PR only] 6-50 LoC changed system [PR only] Infrastructure and configuration for the project
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Migrate content repo to use fred by default
2 participants