forked from scratchfoundation/scratch-editor
-
Notifications
You must be signed in to change notification settings - Fork 1
Closed
Description
Goal
scratch-gui パッケージの ESLint JSDoc ルールによる 23 個の warning をすべて除去し、lint を warning-free にする。
Affected Files
src/containers/ruby-downloader.jsx—@param名が関数引数と不一致(convertedを削除)src/containers/ruby-tab/furigana-renderer.js—@param不足(text,fontSize)src/lib/furigana-annotator.js—@param不足(source,nodex2)src/lib/furigana-call-helpers.js—@param不足(node,extensionName)src/lib/module-sync.js— ブロック説明後の余分な空行src/lib/ruby-generator/data.js—@param/@returns不足 +@rubyエスケープsrc/lib/ruby-generator/face_sensing.js— JSDocコメント不足src/lib/ruby-to-blocks-converter/looks.js—@param不足(converter,arg)src/lib/ruby-to-blocks-converter/node-utils.js—@rubyエスケープsrc/lib/ruby-to-blocks-converter/variables.js—@aエスケープ +Object→object+@param不足 +@rubyエスケープsrc/lib/version-checker.js—@returns説明不足
Implementation Steps
- Phase 1: 全10ファイルのJSDocコメントを修正し、lint warning を 0 にする
@param不足 → 追加@param名不一致 → 削除@ruby/@aインラインタグ → バッククォートでエスケープObject→object(型名小文字化)@returns不足/説明不足 → 追加require-jsdoc→ JSDocコメント追加tag-lines→ 余分な空行削除- lint 確認 → commit & push → PR 作成
Definition of Done
- lint が warning 0 で pass
- CI green
Test Plan
| Type | Timing | Target |
|---|---|---|
| lint | 修正後 | npm run lint で warning 0 を確認 |
| CI | push 後 | 全テストスイートが green |
Risks & Open Questions
なし。すべて機械的なJSDocコメントの修正であり、コードの動作には影響しない。
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
No labels