Add comments

Author: ehmicky

.travis.yml

@@ -6,22 +6,33 @@ os:
   - osx
   - windows
 script: npm run ci-test
+# When pushing a tagged commit, Travis adds two builds: one with the tag, one
+# without. We only want to build the one with the tag, because it's the one
+# that runs the deployment stage
 if: '!(commit_message =~ /^Release/ && tag is blank)'
+# Deployment must happen after tests for the whole matrix have been performed,
+# so we use `jobs.include` with a different `stage`
 jobs:
   include:
     - stage: Deploy
+      # We have already run tests
       script: skip
+      # We want to run install step and keep built files for release
+      skip_cleanup: true
+      # We run `release-it` locally (which tagged the commit and create a
+      # GitHub release) but we publish to npm only after CI success.
       deploy:
         provider: npm
         email: [email protected]
         api_key:
           secure: 'HGXkllgDvBpKufrSY73j9u2QqEm0aiKXpyLQ4DvwPTaOlFnWWe74j+eiNJvpdKoRoi9CZQGFnaO0ngzOUfGgxP5fa0N1Gt5Pjiwr2dcqsKgkN1OMozzkS8Wzzwtpulfs7hUdfydqjo0P+5A9qEPRA1yhn5TFUYompOT1IYZyyS8hJwrn0o516/ZkxzpJI75p3vUv62o+Fnd7RyWeCVKIjzS2lEV3YSsVeixlSFKJ08F7YtlWgwHju5wCkcYvtMH3S1XQz8SFGbnT74XKxb5SnEaIr6j3c4TS6JP0myCZL/t9YlRzPUHCsYRgxAs7IdsHr+eK+SBODHyfzpkBcTErfdyf74AymJu244EFT1jAGMeydq4ZYexrgcftVTPYsdmzSZJgSXoL7ziayXm1qstFmJPQrfcRHag0BEivPHl1nAvS2sstk7YdQQN/1ri39qi9oK24fdL8GQ7kcj9shRHBd2UTZDl8tE/3g0Ujn+iQsUeJ/JFAlaa8hN9SaRGhG/dvdiYAu3lB8ga1aDKNlnHEkR6eR9GCPV8WjbooSNBSHEJC/cGHLzJeeF4By8prshCIBd3iPou6JDtjpyiVa2nOnlFIdU//fDdU0wq6tcp/LxUqYaBpHvU3LpklKVZzUAmipBnlKLgEpgt1sSpuJckTJOY+TWt6T3egYtieLeIiLqk='
-        skip_cleanup: true
         on:
           tags: true
 cache: npm
+# This merges all LCOV files into one before sending to Coveralls, allowing
+# code coverage percentage to include OS/environment-specific coverage.
 env:
   global: COVERALLS_PARALLEL=true
 notifications:
-  email: false
   webhooks: https://coveralls.io/webhook
+  email: false