Skip to content

Adjust build config for watch mode

Sarah German requested to merge fe-copy-check into main

What does this MR do and why?

  • In watch mode, skip asset copy if assets already exist
    • Why: Improves performance of FE build process during development
  • Adjust clean make target to remove copied files and keep node modules
    • Why: clean should remove build artifacts only, it should not remove dependencies

How to set up and validate locally

Numbered steps to set up and validate the change are strongly suggested.

  1. Configure a local GitLab Docs environment: https://gitlab.com/gitlab-org/technical-writing-group/gitlab-docs-hugo/-/blob/main/doc/setup.md.
  2. Run make clean
  3. Run make view, site should load with expected FE assets
  4. Run yarn run dev and make a change to a CSS or JS file. Change should show in-browser; logs should not show an extra copy operation; site should load with expected FE assets

Merge request acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Merge request reports