Skip to content

Refactor how minify script works

Achilleas Pipinellis requested to merge scripts-check-existence into master

This MR refactors the minify process. Namely:

  • Checks the existence of the scripts before running them: Since all Docker builds share the same image and we add new scripts as we go along, we need a way to check if the scripts do really exist. Older branches will not have them and will break otherwise.
  • Fixes minify script to check the right directory
  • Doesn't minify XML files anymore: The only XML file is the sitemap, no need to minify that. It also "breaks" when viewing it through the browser.

I discovered this while investigating #480 (closed), and the single image job for 12.5 failed https://gitlab.com/gitlab-org/gitlab-docs/-/jobs/373219702.

Merge request reports