Skip to content

Commit

Permalink
Apply suggestions from code review
Browse files Browse the repository at this point in the history
Co-authored-by: Chengzhong Wu <legendecas@gmail.com>
  • Loading branch information
GeoffreyBooth and legendecas authored Feb 5, 2024
1 parent 4008c65 commit dfdf6a7
Showing 1 changed file with 4 additions and 3 deletions.
7 changes: 4 additions & 3 deletions lib/internal/vm.js
Original file line number Diff line number Diff line change
Expand Up @@ -42,8 +42,8 @@ function isContext(object) {

/**
* Computes the host-defined option ID based on the provided importModuleDynamically and hint.
* @param {import('internal/modules/esm/utils').ImportModuleDynamicallyCallback | symbol | undefined} importModuleDynamically -
* The importModuleDynamically function or undefined.
* @param {import('internal/modules/esm/utils').ImportModuleDynamicallyCallback
* | symbol | undefined} importModuleDynamically - The importModuleDynamically function or undefined.
* @param {string} hint - The hint for the option ID.
* @returns {symbol} - The host-defined option id
* ID.
Expand Down Expand Up @@ -165,7 +165,8 @@ function internalCompileFunction(
* @param {boolean} produceCachedData - Indicates whether to produce cached data.
* @param {object} parsingContext - The parsing context of the script.
* @param {symbol} hostDefinedOptionId - The host-defined option ID.
* @param {import('internal/modules/esm/utils').ImportModuleDynamicallyCallback} importModuleDynamically - Indicates whether to import modules dynamically.
* @param {import('internal/modules/esm/utils').ImportModuleDynamicallyCallback} importModuleDynamically -
* Indicates whether to import modules dynamically.
* @returns {ContextifyScript} The created contextify script.
*/
function makeContextifyScript(code,
Expand Down

0 comments on commit dfdf6a7

Please sign in to comment.