diff options
author | Roeland Jago Douma <roeland@famdouma.nl> | 2017-03-10 10:45:27 +0100 |
---|---|---|
committer | Roeland Jago Douma <roeland@famdouma.nl> | 2017-03-22 08:49:03 +0100 |
commit | 12de9ad6550bed856980e51b6a6fce5d38d44fbf (patch) | |
tree | afb7f1fece68c3e5efe8133fe767fa51ff0a404d | |
parent | 98a14909e1ed79a62f9f3e733782cd5b0f949339 (diff) | |
download | nextcloud-server-12de9ad6550bed856980e51b6a6fce5d38d44fbf.tar.gz nextcloud-server-12de9ad6550bed856980e51b6a6fce5d38d44fbf.zip |
Add CI step to verify merged vendor js
Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
-rw-r--r-- | .drone.yml | 8 | ||||
-rwxr-xr-x | build/mergejschecker.sh | 26 |
2 files changed, 34 insertions, 0 deletions
diff --git a/.drone.yml b/.drone.yml index 5d4ab2ca963..5cfba60cd45 100644 --- a/.drone.yml +++ b/.drone.yml @@ -19,6 +19,13 @@ pipeline: when: matrix: TESTS: check-autoloader + check-mergejs: + image: nextcloudci/php7.0:php7.0-7 + commands: + - bash ./build/mergejschecker.sh + when: + matrix: + TESTS: check-mergejs app-check-code: image: nextcloudci/php7.0:php7.0-7 commands: @@ -493,6 +500,7 @@ matrix: - TESTS: integration-ldap-features - TESTS: jsunit - TESTS: check-autoloader + - TESTS: check-mergejs - TESTS: app-check-code - TESTS: syntax-php5.6 - TESTS: syntax-php7.0 diff --git a/build/mergejschecker.sh b/build/mergejschecker.sh new file mode 100755 index 00000000000..a8c5c37a1f1 --- /dev/null +++ b/build/mergejschecker.sh @@ -0,0 +1,26 @@ +#!/usr/bin/env bash + +#Regenerate the vendors core.js +echo +echo "Regenerating core/vendor/core.js" + +d=`dirname $(readlink -f $0)` + +php $d/mergejs.php + +files=`git diff --name-only` + +for file in $files +do + if [[ $file == core/vendor/core.js ]] + then + echo "The merged vendor file is not up to date" + echo "Please run: php build/mergejs.php" + echo "And commit the result" + exit 1 + break + fi +done + +echo "Vendor js merged as expected. Carry on" +exit 0 |