From 44eacd148bf08977220a1fc101fb74c7a3a8c5f1 Mon Sep 17 00:00:00 2001 From: jackspirou Date: Sun, 24 Jan 2016 08:46:03 -0600 Subject: [PATCH] feat: adding build number env var for compatibility with drone and other CI systems s more than one blank line not allowed 38:2 s trailing spaces not allowed s trailing spaces not allowed --- README.md | 5 ++++- lib/sauce_launcher.js | 9 ++++++--- 2 files changed, 10 insertions(+), 4 deletions(-) diff --git a/README.md b/README.md index 9f095ce..ae63d89 100644 --- a/README.md +++ b/README.md @@ -116,10 +116,13 @@ Options to send to Sauce Connect. Check [here](https://github.com/bermi/sauce-co ### build Type: `String` Default: *One of the following environment variables*: -`process.env.TRAVIS_BUILD_NUMBER` `process.env.BUILD_NUMBER` `process.env.BUILD_TAG` +`process.env.CI_BUILD_NUMBER` +`process.env.CI_BUILD_TAG` +`process.env.TRAVIS_BUILD_NUMBER` `process.env.CIRCLE_BUILD_NUM` +`process.env.DRONE_BUILD_NUMBER` ID of the build currently running. This should be set by your CI. diff --git a/lib/sauce_launcher.js b/lib/sauce_launcher.js index 91209f0..af7c9f9 100644 --- a/lib/sauce_launcher.js +++ b/lib/sauce_launcher.js @@ -27,10 +27,13 @@ function processConfig (helper, config, args) { tunnelIdentifier: tunnelIdentifier }) - var build = process.env.TRAVIS_BUILD_NUMBER || - process.env.BUILD_NUMBER || + var build = process.env.BUILD_NUMBER || process.env.BUILD_TAG || - process.env.CIRCLE_BUILD_NUM + process.env.CI_BUILD_NUMBER || + process.env.CI_BUILD_TAG || + process.env.TRAVIS_BUILD_NUMBER || + process.env.CIRCLE_BUILD_NUM || + process.env.DRONE_BUILD_NUMBER var defaults = { version: '',