Build tasks: Rename compile to build:app

[colinrotherham]

We've planned to remove --destination flag to simplify our build tasks in:

• Refactor file copying tasks to remove Gulp #2711
• Split project into separate app, config, lib and tasks packages #2886

But we currently have one build script that has some overlap:

npx gulp compile
npx gulp build:package --destination "package"
npx gulp build:dist --destination "dist"

Surprise: The compile task is also run on build:dist and unnecessarily generates documentation

Changes

This PR does three things:

1. Renames Gulp compile task to build:app (the review app)
2. Renames npm build:compile script to build:app
3. Prevents build:dist from building JSDoc and SassDoc

This lines up all our tasks so we can remove --destination in future

Before

npx gulp compile

After

npx gulp build:app --destination "public"

[36degrees]

Looks good, bar one minor comment on some of the docs changes

[36degrees]

Are these the wrong way round?

Suggested change

	- run sub tasks from `gulp styles` without ESLint code quality checks
	- run sub tasks from `gulp scripts` without StyleLint code quality checks
	- run sub tasks from `gulp styles` without StyleLint code quality checks
	- run sub tasks from `gulp scripts` without ESLint code quality checks

[colinrotherham]

@36degrees Well spotted, they've been switched since 4c11f65 🤦‍♂️

✅ Fixed