Browse Source

Merge pull request #6879 from telamonian/fix-remove-package-cmd

fixed silent error in `usage` CI test by fixing `jlpm remove:package`
Ian Rose 5 years ago
parent
commit
08032d941c
2 changed files with 3 additions and 3 deletions
  1. 2 2
      buildutils/src/remove-package.ts
  2. 1 1
      scripts/travis_script.sh

+ 2 - 2
buildutils/src/remove-package.ts

@@ -37,5 +37,5 @@ if (!fs.existsSync(packagePath)) {
 // Remove the package from the local tree.
 fs.removeSync(path.dirname(packagePath));
 
-// Update the core jupyterlab build dependencies.
-utils.run('npm run integrity');
+// Remove any dependencies on the package (will also run `jlpm integrity`)
+utils.run(`jlpm remove:dependency ${target}`);

+ 1 - 1
scripts/travis_script.sh

@@ -180,7 +180,7 @@ if [[ $GROUP == usage ]]; then
     jlpm run build:packages
     jlpm run build:dev
     python -m jupyterlab.browser_check --dev-mode
-    rm -rf packages/foo
+    jlpm run remove:package foo
     jlpm run integrity
 
     ## Test app directory support being a symlink