Skip to content

Commit

Permalink
docs: reword the assumptions on package manager
Browse files Browse the repository at this point in the history
  • Loading branch information
byCedric committed Mar 17, 2022
1 parent eb1ffb8 commit 459b567
Show file tree
Hide file tree
Showing 2 changed files with 8 additions and 6 deletions.
7 changes: 4 additions & 3 deletions build/setup/index.js
Original file line number Diff line number Diff line change
Expand Up @@ -67043,9 +67043,10 @@ async function resolvePackage(name, range) {
}
exports.resolvePackage = resolvePackage;
/**
* Install a module using a node package manager.
* The installation is executed in a temporary folder.
* After this is complete, it's "installed" into the worker's tool cache.
* Install a module using a node package manager, inside a temporary directory.
* If that's successful, move the module into the worker's tool cache.
*
* Note, we do assume that the packager is globally available AND has the `add <pkgspec>` command.
*/
async function installPackage(name, version, manager) {
const temp = (0, worker_1.tempPath)(name, version);
Expand Down
7 changes: 4 additions & 3 deletions src/packager.ts
Original file line number Diff line number Diff line change
Expand Up @@ -31,9 +31,10 @@ export async function resolvePackage(name: string, range: string): Promise<strin
}

/**
* Install a module using a node package manager.
* The installation is executed in a temporary folder.
* After this is complete, it's "installed" into the worker's tool cache.
* Install a module using a node package manager, inside a temporary directory.
* If that's successful, move the module into the worker's tool cache.
*
* Note, we do assume that the packager is globally available AND has the `add <pkgspec>` command.
*/
export async function installPackage(name: string, version: string, manager: string) {
const temp = tempPath(name, version);
Expand Down

0 comments on commit 459b567

Please sign in to comment.