Unexpected major release because commit description includes a line starting with "Breaking change"
まだ誰も着手していません。
評価
- 難易度
- 3/5
- 見積もり時間
- 1〜2日
- 初心者へのやさしさ
- 35/100
- issue の種類
- バグ
- 明瞭さ
- おおむね明確
- 活発さ
- 停滞
- 技術スタック
- javascript
- 領域
- release
調査の方向性
test/integration.test.js と、issue に示されている analyzeCommits の呼び出しから始めます。提供されたコミットメッセージを再現し、その説明がどのように分類されるかを調べます。回帰テストでそのケースがカバーされ、minor リリースが返され、既存の統合テストが引き続きパスすれば完了です。
索引モデルが issue の本文から書いたものです。
説明
It's possible there isn't a reasonable way to avoid this, but I recently encountered an issue where a commit message triggered a unexpected major release for me.
diff --git a/test/integration.test.js b/test/integration.test.js
index 112bc3a..550f9ee 100644
--- a/test/integration.test.js
+++ b/test/integration.test.js
@@ -25,6 +25,24 @@ test('Parse with "conventional-changelog-angular" by default', async (t) => {
t.true(t.context.log.calledWith("Analysis of %s commits complete: %s release", 2, "minor"));
});
+test('Parse with "conventional-changelog-angular" by default 2', async (t) => {
+ const commits = [
+ {
+ hash: "456",
+ message: `feat: new feature
+
+ Breaking change for unsupported use case but not a breaking API change
+
+ This isn't being released as major because the breaking change
+ is to an unsupported/undocumented use, but we still want to notify
+ people that might be using it that way.`,
+ },
+ ];
+ const releaseType = await analyzeCommits({}, { cwd, commits, logger: t.context.logger });
+
+ t.is(releaseType, "minor");
+});
+
test('Accept "preset" option', async (t) => {
const commits = [
{ hash: "123", message: "Fix: First fix (fixes #123)" },
There may not be a reasonable way to prevent this without breaking intended use, but when combined with https://github.com/semantic-release/semantic-release/issues/3721 this led to quite the predicament.
- 主要言語
- JavaScript
- スター
- 448
- フォーク
- 82
- 平均マージ
- 17時間 41分
- マージ済み PR（30日）
- 4
コントリビューションガイド
このリポジトリのコントリビューションガイドは索引されていません
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
semantic-release/commit-analyzer のほかの issue
-
released on @beta
難易度 2/5 1〜3時間 初心者へのやさしさ 65/100
semantic-release/commit-analyzer#921 · コメント 8 件 · リアクション 7 件 ·
-
難易度 3/5 1〜2日 初心者へのやさしさ 42/100
semantic-release/commit-analyzer#805 · コメント 6 件 ·
-
難易度 4/5 3〜5日 初心者へのやさしさ 25/100
semantic-release/commit-analyzer#802 · コメント 1 件 ·
-
released on @beta
難易度 3/5 1〜2日 初心者へのやさしさ 55/100
semantic-release/commit-analyzer#759 · コメント 13 件 · リアクション 6 件 ·
-
難易度 2/5 1〜3時間 初心者へのやさしさ 35/100
semantic-release/commit-analyzer#750 · コメント 1 件 ·
semantic-release/commit-analyzer の issue をすべて見る
似ている issue
-
Bug
難易度 2/5 1〜3時間 初心者へのやさしさ 70/100
Automattic/safe-publish#594 ·
-
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
dream-num/dsh-univer-office#104 ·
-
comp/dashboard invalid P3
難易度 2/5 1〜3時間 初心者へのやさしさ 70/100
NousResearch/hermes-agent#121143 ·
-
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
avniproject/avni-webapp#1811 ·
-
area/auroraboot area/webui bug
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100