test [publish binary]

Author: 3np

.travis.yml

@@ -77,12 +77,6 @@ matrix:
         - ./scripts/publish.sh --toolset=${TOOLSET:-} --debug=$([ "${BUILDTYPE}" == 'debug' ] && echo "true" || echo "false")
       script:
         - true
-      # Overrides `install` to avoid initializing clang toolchain
-      install:
-        - make ${BUILDTYPE}
-      # Overrides `script` to disable publishing
-      before_script:
-        - npm test
     # Sanitizer build node v12/Debug
     - os: linux
       env: BUILDTYPE=debug TOOLSET=asan CXXFLAGS="-fext-numeric-literals"
@@ -108,12 +102,6 @@ matrix:
         - ./scripts/publish.sh --toolset=${TOOLSET:-} --debug=$([ "${BUILDTYPE}" == 'debug' ] && echo "true" || echo "false")
       script:
         - true
-      # Overrides `install` to avoid initializing clang toolchain
-      install:
-        - make ${BUILDTYPE}
-      # Overrides `script` to disable publishing
-      before_script:
-        - npm test
     # Coverage build
     - os: linux
       env: BUILDTYPE=debug CXXFLAGS="--coverage" LDFLAGS="--coverage"

scripts/publish.sh

@@ -28,27 +28,22 @@ function is_pr_merge() {
 # `publish` is used to publish binaries to s3 via commit messages if:
 # - the commit message includes [publish binary]
 # - the commit message includes [republish binary]
-# - the commit is not a pr_merge (checked with `is_pr_merge` function)
 function publish() {
   echo "dumping binary meta..."
   ./node_modules/.bin/node-pre-gyp reveal --loglevel=error $@
 
   echo "determining publishing status..."
 
-  if [[ $(is_pr_merge) ]]; then
-      echo "Skipping publishing because this is a PR merge commit"
-  else
-      echo "Commit message: ${COMMIT_MESSAGE}"
+  echo "Commit message: ${COMMIT_MESSAGE}"
 
-      if [[ ${COMMIT_MESSAGE} =~ "[publish binary]" ]]; then
-          echo "Publishing"
-          ./node_modules/.bin/node-pre-gyp package publish $@
-      elif [[ ${COMMIT_MESSAGE} =~ "[republish binary]" ]]; then
-          echo "Re-Publishing"
-          ./node_modules/.bin/node-pre-gyp package unpublish publish $@
-      else
-          echo "Skipping publishing since we did not detect either [publish binary] or [republish binary] in commit message"
-      fi
+  if [[ ${COMMIT_MESSAGE} =~ "[publish binary]" ]]; then
+    echo "Publishing"
+    ./node_modules/.bin/node-pre-gyp package publish $@
+  elif [[ ${COMMIT_MESSAGE} =~ "[republish binary]" ]]; then
+    echo "Re-Publishing"
+    ./node_modules/.bin/node-pre-gyp package unpublish publish $@
+  else
+    echo "Skipping publishing since we did not detect either [publish binary] or [republish binary] in commit message"
   fi
 }