commitizen / commitizen/cz-cli

regression: Pnpm `wrap-ans` require() of ES Module error

オープン
#996 コメント 0 件 リアクション 0 件 担当者 0 名 GitHub で見る
主要言語
JavaScript
スター
17.5k
フォーク
566
平均マージ
8時間 16分
マージ済み PR(30日)
1

説明

### Description
`cz-cli` dependency `wrap-ans` does not import properly, causing error when running `pnpm commit`

**package.json**
```Json
{
"type": "module",
"scripts": {
"commit": "cz",
...
},
...
}
```

**error in cli**
```Bash
require() of ES Module /Users/menglinmaker/Documents/2-Engineering/OpenSource/npm-library-boilerplate/node_modules/wrap-ansi/node_modules/string-width/index.js from /Users/menglinmaker/Documents/2-Engineering/OpenSource/npm-library-boilerplate/node_modules/wrap-ansi/index.js not supported.
Instead change the require of /Users/menglinmaker/Documents/2-Engineering/OpenSource/npm-library-boilerplate/node_modules/wrap-ansi/node_modules/string-width/index.js in /Users/menglinmaker/Documents/2-Engineering/OpenSource/npm-library-boilerplate/node_modules/wrap-ansi/index.js to a dynamic import() which is available in all CommonJS modules.
```

Note: using `"type": "commonjs"` leads to the same result

### Additional details
This issue only occurs on `commitizen@4.2.5` and after.
I found `commitizen@4.2.2` to behave exactly how I liked.

コントリビューションガイド

コントリビューションガイドを開く

評価

この issue はまだ評価されていません。

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。