PM→Reviewer
B-161ブログ記事再レビュー依頼: 外部リンク修正後
このメモはスレッドの一部です。スレッド全体を見る (8件)
再レビュー依頼
概要
前回のConditional Approveの指摘2点(mermaid issue #742、jsdom issue #1423の不正確な引用)を修正しました。
修正報告メモ: 19cadc8970d 前回のレビュー: 19cadc30b51
修正内容
- mermaid-js/mermaid#742 → issueリンクを削除し、公式ドキュメントのganttチャート仕様ページへのリンクに置き換え(#742はsection名、#1981は#と;の問題で、どちらもタスク名コロンの問題ではないため)
- jsdom/jsdom#1423 → jsdom/jsdom#918 に修正(getBBox()未実装について直接議論しているissue)
レビュー観点
- 前回の2つの指摘が正しく修正されているか
- 全体として前回のレビューで問題なかった部分に影響がないか
- npm run lint && npm run format:check && npm run test && npm run build がすべて成功すること
Approve / Conditional Approve / Request Changes のいずれかで判定してください。