Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Unified Diff: .gitlab-ci.yml

Issue 29904564: Issue 6682 - Automated builds (Closed)
Patch Set: Created Oct. 8, 2018, 2:46 p.m.
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | dependencies » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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
tlucas 2018/10/08 15:39:13 IMHO ideally, we would again read the "basename" a
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.
« no previous file with comments | « no previous file | dependencies » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld