temporarily turn off yarn (ampproject#8356)

dewen · Mar 23, 2017 · d79cf56 · d79cf56
1 parent fd822fb
commit d79cf56
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 7,940 deletions.
diff --git a/.travis.yml b/.travis.yml
@@ -37,6 +37,6 @@ env:
     - NPM_CONFIG_PROGRESS="false"
 
 cache:
-  yarn: true
+  #yarn: true
   directories:
   - node_modules
diff --git a/build-system/pr-check.js b/build-system/pr-check.js
@@ -207,12 +207,13 @@ function main(argv) {
     return 0;
   }
 
-  if (files.includes('package.json') ?
-        !files.includes('yarn.lock') : files.includes('yarn.lock')) {
-    console.error('pr-check.js - any update to package.json or yarn.lock ' +
-        'must include the other file. Please update through yarn.');
-    process.exit(1);
-  }
+  // NOTE(erwinm, #8356): temporary removal to get around travis bug for yarn
+  //if (files.includes('package.json') ?
+        //!files.includes('yarn.lock') : files.includes('yarn.lock')) {
+    //console.error('pr-check.js - any update to package.json or yarn.lock ' +
+        //'must include the other file. Please update through yarn.');
+    //process.exit(1);
+  //}
 
   const sortedBuildTargets = [];
   for (const t of buildTargets) {