fix(transformers): errors.map is not a function

[sweetliquid]

↪️ Pull Request

💻 Examples

🚨 Test instructions

✔️ PR Todo

• Added/updated unit tests for this change
• Filled out test instructions (In case there aren't any unit tests)
• Included links to related issues/PRs

[height]

Link Height tasks by mentioning a task ID in the pull request title or commit messages, or description and comments with the keyword link (e.g. "Link T-123").

💡Tip: You can also use "Close T-X" to automatically close a task when the pull request is merged.

[Shinyaigeek]

https://github.com/parcel-bundler/parcel/blob/v2/packages/transformers/html/src/dependencies.js#L117

The errors collectDependencies throw seems to be only array, so errors.map should be a valid function. Is there some case that parcel throws “errors.map is not a function” error in HTMLTransformer? ( if there is, please share reproduction)

[sweetliquid]

The error is thrown by other functions called by collectDependencies like QUERIES[i].select in browserslist package

throw new BrowserslistError(
  'Unknown version ' + version + ' of ' + name
)

[sweetliquid]

I think errors those are not directly thrown by collectDependencies can be ignored(re-thown).

[sweetliquid]

If I create the .browserslistrc file with any wrong content like: xxx, it thrown emit error: TypeError: errors.map is not a function when I run parcel build

[Shinyaigeek]

Oh I see, I cannot reproduce it but I feel this can happen and this change is reasonable. Thank you !