Only execute "coveralls" toxenv on master once

nicoddemus · nicoddemus · commit 6a98fafe509c · 2016-11-24T16:08:51.000-02:00
Just noticed that the "coveralls" env was being execute after each env. This was introduced by mistake in #2056
diff --git a/appveyor.yml b/appveyor.yml
@@ -6,6 +6,8 @@ environment:
     # https://www.appveyor.com/docs/build-configuration#secure-variables
 
   matrix:
+  # coveralls is not in the default env list
+  - TOXENV: "coveralls"
   # note: please use "tox --listenvs" to populate the build matrix below
   - TOXENV: "linting"
   - TOXENV: "py26"
@@ -36,7 +38,6 @@ install:
 build: false  # Not a C# project, build stuff at the test step instead.
 
 test_script:
+  # if TOXENV is coveralls but %COVERALLS_REPO_TOKEN% is not defined (PRs or forks), skip it
+  - if "%TOXENV%:%COVERALLS_REPO_TOKEN%" == "coveralls:" exit /b 0
   - C:\Python35\python -m tox
-  # coveralls is not in tox's envlist, plus for PRs the secure variable
-  # is not defined so we have to check for it
-  - if defined COVERALLS_REPO_TOKEN C:\Python35\python -m tox -e coveralls
