GeezerVison is showing github.com — lib: fix typos in esm loader comments · nodejs/node@12c5154
File tree
Expand file treeCollapse file tree
Open diff view settings
Expand file treeCollapse file tree
Open diff view settings
Collapse file
Copy file name to clipboardExpand all lines: lib/internal/modules/esm/loader.js +3-3Lines changed: 3 additions & 3 deletions | Original file line number | Diff line number | Diff line change |
|---|
@@ -286,16 +286,16 @@ class ModuleLoader { |
286 | 286 | let job = this.loadCache.get(url, kImplicitTypeAttribute); |
287 | 287 | // This module job is already created: |
288 | 288 | // 1. If it was loaded by `require()` before, at this point the instantiation |
289 | | - // is already completed and we can check the whether it is in a cycle |
290 | | - // (in that case the module status is kEvaluaing), and whether the |
| 289 | + // is already completed and we can check whether it is in a cycle |
| 290 | + // (in that case the module status is kEvaluating), and whether the |
291 | 291 | // required graph is synchronous. |
292 | 292 | // 2. If it was loaded by `import` before, only allow it if it's already evaluated |
293 | 293 | // to forbid cycles. |
294 | 294 | // TODO(joyeecheung): ensure that imported synchronous graphs are evaluated |
295 | 295 | // synchronously so that any previously imported synchronous graph is already |
296 | 296 | // evaluated at this point. |
297 | 297 | // TODO(joyeecheung): add something similar to CJS loader's requireStack to help |
298 | | - // debugging the the problematic links in the graph for import. |
| 298 | + // debugging the problematic links in the graph for import. |
299 | 299 | debug('importSyncForRequire', parent?.filename, '->', filename, job); |
300 | 300 | if (job !== undefined) { |
301 | 301 | mod[kRequiredModuleSymbol] = job.module; |
|