Issue 6682 - Automated builds

.gitlab-ci.yml

Index: .gitlab-ci.yml
diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml
index 2c7a3fb2463a7563268e3d4f5aadd97a48806eca..79429cf0ac248d5fc16313af17423681a2240d86 100644
--- a/.gitlab-ci.yml
+++ b/.gitlab-ci.yml
@@ -22,6 +22,7 @@
 stages:
   - prepare
   - test
+  - build
 
 .dependencies:
   script: &dependencies
@@ -30,15 +31,31 @@ stages:
     - python ensure_dependencies.py
     - "[ -d node_modules ] || npm install"
 
-.test_template:
-  stage: test
-  before_script: *dependencies
+.pull_cache:
   cache:
     key: cache_$CI_COMMIT_SHA
     paths:
       - ./
     policy: pull
 
+.test_template:
+  extends: .pull_cache
+  stage: test
+  before_script: *dependencies
+
+.build:
+  extends: .pull_cache
+  before_script: *dependencies
+  stage: build
+  only:
+    - master
+  script:
+    - python build.py build -t $PLATFORM
+  artifacts:
+    paths:
+      - ./*.xpi
+      - ./*.zip
+      - ./*.appx

[Sebastian Noack, 2018/10/08 15:16:55]

Perhaps we should make the filename a little more specific. e.g.
adblockpluschrome-*.zip. Also from the example in the documentation adding ./
doesn't seem to be necessary, or do I miss something here?

[tlucas, 2018/10/08 15:39:13]

IMHO ideally, we would again read the "basename" and the extension for $PLATFORM
here, to match to correct file - unfortunately, (dynamic) variables are not
available in artifact:paths.

-> Done, as per your suggestion.

 
 prepare-dependencies:
   stage: prepare
@@ -63,3 +80,19 @@ lint:
   extends: .test_template
   script:
     - npm run lint
+
+build:chrome:
+  extends: .build
+  variables:
+    PLATFORM: chrome
+
+build:gecko:
+  extends: .build
+  variables:
+    PLATFORM: gecko
+
+build:edge:
+  extends: .build
+  variables:
+    PLATFORM: edge
+  allow_failure: true

[Sebastian Noack, 2018/10/08 15:16:55]

Can we add a comment referring to the issue here?

[tlucas, 2018/10/08 15:39:13]

Done.