Skip to content

Commit b6154a1

Browse files
authored
Merge pull request #9613 from keymanapp/chore/web/clean-publish-folder
chore(web): builds that output to web/build/publish should also clean it
2 parents 5455aa7 + 4f82264 commit b6154a1

File tree

2 files changed

+12
-2
lines changed

2 files changed

+12
-2
lines changed

web/src/app/browser/build.sh

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -41,6 +41,11 @@ builder_describe_outputs \
4141

4242
#### Build action definitions ####
4343

44+
do_clean() {
45+
rm -rf "$KEYMAN_ROOT/web/build/$SUBPROJECT_NAME"
46+
rm -rf "$KEYMAN_ROOT/web/build/publish"
47+
}
48+
4449
compile_and_copy() {
4550
local COMPILE_FLAGS=
4651
if builder_has_option --ci; then
@@ -61,7 +66,7 @@ compile_and_copy() {
6166
}
6267

6368
builder_run_action configure verify_npm_setup
64-
builder_run_action clean rm -rf "$KEYMAN_ROOT/web/build/$SUBPROJECT_NAME"
69+
builder_run_action clean do_clean
6570
builder_run_action build compile_and_copy
6671

6772
# No headless tests for this child project. Currently, DOM-based unit &

web/src/app/ui/build.sh

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -40,6 +40,11 @@ builder_describe_outputs \
4040

4141
#### Build action definitions ####
4242

43+
do_clean() {
44+
rm -rf "$KEYMAN_ROOT/web/build/$SUBPROJECT_NAME"
45+
rm -rf "$KEYMAN_ROOT/web/build/publish"
46+
}
47+
4348
compile_and_copy() {
4449
compile $SUBPROJECT_NAME
4550

@@ -51,7 +56,7 @@ compile_and_copy() {
5156
}
5257

5358
builder_run_action configure verify_npm_setup
54-
builder_run_action clean rm -rf "$KEYMAN_ROOT/web/build/$SUBPROJECT_NAME"
59+
builder_run_action clean do_clean
5560
builder_run_action build compile_and_copy
5661

5762
# No headless tests for this child project. Currently, DOM-based unit &

0 commit comments

Comments
 (0)