mdn / yari

The platform code behind MDN Web Docs
Mozilla Public License 2.0
1.16k stars 486 forks source link

fix(package): run render:html on prepack #11368

Closed caugner closed 6 days ago

caugner commented 6 days ago

Summary

Fixes #11365.

Problem

Since https://github.com/mdn/yari/pull/10953, yarn tool spas no longer builds index.html files, and these are instead built via yarn render:html, but this command isn't run when packaging yari for npm.

Solution

Run yarn render:html as part of the prepack script.


How did you test this change?

Fixed the npm-published-simulation.yml workflow to actually fail when http://localhost:5042/en-US/ returns an HTTP error.