commitizen / commitizen/cz-cli
Let the adapters depend on inquirer
- 主要言語
- JavaScript
- スター
- 17.5k
- フォーク
- 566
- 平均マージ
- 8時間 16分
- マージ済み PR(30日)
- 1
説明
Instead of us supplying inquirer to the adapters, I think that it would be very nice if they could depend on it themselves. Then they can update that dependency at their own discretion instead of being dependent on the main module.
Currently you can choose to be compatible with either commitizen > 2.8.0, or with versions < 2.8.0 since we upgraded inquirer which broke the api. Since this has already been updated as well in conventional-changelog, I think it's to late to revert the update here.
The problem I'm facing as an adapter author is that if I upgrade, I need to go around and tell everyone at the office to update to a newer version of commitizen. I also must upgrade all of the repos that uses our internal adapter at the same time. Otherwise we will have some repos that works in one version, and some that works in other.
Basically this is quite a lot of pain and I think that we should move towards letting the adapters depend on their dependencies separately.
Also, there might even be a benefit to let our interface to the adapters be completely language agnostic. We could just execute the file, and use the output as the commit message.
@jimthedev I know that you are very busy but if you could just quickly answer if you are in favour of this, then I could implement it 👌
コントリビューションガイド
調査の方向性
ファイルやテストは指定されていません。まず、cz-cli が adapters に inquirer をどのように渡しているか、また adapters の出力がどのように commit message になるかを追跡します。実装範囲を決定する前に、adapters の依存関係の境界と、言語に依存しない実行動作を定義します。
索引モデルが issue の本文から書いたものです。
評価
- 技術スタック
- javascript, node.js
- 領域
- cli, developer-experience
- issue の種類
- 機能追加
- 難易度
- 5/5
- 見積もり時間
- 1週間以上
- 活発さ
- 停滞
- 明瞭さ
- 説明が足りない
- 初心者へのやさしさ
- 25/100