nodejs / nodejs/core-validate-commit
Invalid Commit Message
未關閉
還沒有人認領這個 Issue。
- 主要語言
- JavaScript
- 星號
- 55
- 分支
- 62
- 平均合併
- 4 天 9 小時
- 30 天內合併 PR
- 1
描述
This PR was merged with a non-imperative verb starting the commit message: benchmark: added new benchmarks for blob.
From what the docs says, it should be a invalid message.
The CI did raised an error about the length (that I fix) but didn't raise an error about this verb and it was merged.
貢獻指南
從這裡開始
- 先讀完整個 Issue，再讀專案的貢獻指南。
- 在 Issue 下留言說明你要接手 —— 這能避免兩個人做同樣的事。
- Fork 儲存庫，在一個分支上完成修改。
- 送出 Pull Request，並在描述裡引用這個 Issue 編號。
研究方向
先從提交訊息指南以及 PR 49730 所執行的驗證開始，接著追蹤回報訊息長度錯誤的檢查。確認提交主旨開頭的動詞是如何被評估的。完成的標準是，像「added new benchmarks for blob」這樣的非祈使句主旨會被 CI 拒絕，而有效的祈使句措辭能夠通過。
由索引模型根據 Issue 內容生成。
評估
- 技術堆疊
- javascript
- 領域
- ci-cd, testing-qa
- Issue 類型
- 缺陷
- 難度
- 3/5
- 預估耗時
- 1-2 天
- 活躍度
- 停滯
- 描述清晰度
- 基本清楚
- 新手友好度
- 35/100