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-23 13:40:30 +0100 |
commit | 588f47d498e78b027fd635ebfaa47876ec8e21b7 (patch) | |
tree | 1e96f5ae87078e182af09bd63f0233000fd187b2 | |
parent | eda98ed3df5f304f277f651941cc9ab19dee07bb (diff) | |
download | nextcloud-server-588f47d498e78b027fd635ebfaa47876ec8e21b7.tar.gz nextcloud-server-588f47d498e78b027fd635ebfaa47876ec8e21b7.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 | 25 |
2 files changed, 33 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..aee2629956b --- /dev/null +++ b/build/mergejschecker.sh @@ -0,0 +1,25 @@ +#!/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" + break + fi +done + +echo "Vendor js merged as expected. Carry on" +exit 0 |