guardian / frontend

The Guardian DotCom.
https://theguardian.com
Other
5.84k stars 555 forks source link

All task runner `require` calls are static #27255

Closed mxdvl closed 3 months ago

mxdvl commented 3 months ago

What is the value of this and can you measure success?

Paves the way for the upcoming ESM migration.

What does this change?

Extract dynamic require calls to top-level static ones.

github-actions[bot] commented 3 months ago

Deploy build 2220 of dotcom:frontend-all to CODE

All deployment options - [Deploy build 2220 of `dotcom:frontend-all` to CODE](https://riffraff.gutools.co.uk/deployment/deployAgain?project=dotcom%3Afrontend-all&build=2220&stage=CODE&updateStrategy=MostlyHarmless&action=deploy) - [Deploy parts of build 2220 to CODE by previewing it first](https://riffraff.gutools.co.uk/preview/yaml?project=dotcom%3Afrontend-all&build=2220&stage=CODE&updateStrategy=MostlyHarmless) - [What's on CODE right now?](https://riffraff.gutools.co.uk/deployment/history?projectName=dotcom%3Afrontend-all&stage=CODE)

From guardian/actions-riff-raff.

prout-bot commented 3 months ago

Seen on FRONTS-PROD, ADMIN-PROD (merged by @mxdvl 10 minutes and 57 seconds ago)