Skip to content

Comments

Rename misleading coverage variables in jsdoc-validation workflow#419

Merged
pethers merged 2 commits intostricttypingfrom
copilot/sub-pr-407-again
Feb 21, 2026
Merged

Rename misleading coverage variables in jsdoc-validation workflow#419
pethers merged 2 commits intostricttypingfrom
copilot/sub-pr-407-again

Conversation

Copy link
Contributor

Copilot AI commented Feb 21, 2026

TOTAL_JS and FILES_WITH_JSDOC were inaccurate names since the step counts both .js and .ts files and detects any /** */ doc comments, not strictly JSDoc.

Changes

  • TOTAL_JSTOTAL_SOURCE_FILES
  • FILES_WITH_JSDOCFILES_WITH_DOC_COMMENTS
TOTAL_SOURCE_FILES=$(find src/browser scripts \( -name "*.js" -o -name "*.ts" \) | ...)
FILES_WITH_DOC_COMMENTS=$(find src/browser scripts \( -name "*.js" -o -name "*.ts" \) ... -exec grep -l "/\*\*" {} \; | wc -l)

echo "📈 Documentation Coverage: $FILES_WITH_DOC_COMMENTS / $TOTAL_SOURCE_FILES files"

💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

…C_COMMENTS

Co-authored-by: pethers <1726836+pethers@users.noreply.github.com>
Copilot AI changed the title [WIP] Address feedback on evening analysis content validator Rename misleading coverage variables in jsdoc-validation workflow Feb 21, 2026
Copilot AI requested a review from pethers February 21, 2026 21:32
@pethers pethers marked this pull request as ready for review February 21, 2026 21:43
@pethers pethers merged commit 8d8e761 into stricttyping Feb 21, 2026
@pethers pethers deleted the copilot/sub-pr-407-again branch February 21, 2026 21:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants