Skip to content

Conversation

@0xLeif
Copy link
Contributor

@0xLeif 0xLeif commented Jan 5, 2026

Summary

  • Converts documentation comments with 3+ consecutive /// lines to /** */ block style
  • Files updated: Terminal.swift, Logger.swift, ITerm2Image.swift, KittyImage.swift, Terminal+Input.swift
  • Maintains consistency with CorvidLabs documentation style guidelines

Test Plan

  • Build passes locally
  • No functional changes

🤖 Generated with Claude Code

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR standardizes documentation comments across the codebase by converting triple-slash comments with 3 or more consecutive lines to block comment style, aligning with CorvidLabs documentation guidelines.

  • Converts 5 multi-line documentation comments from /// to /** */ block style
  • Maintains the project's established block comment format (no asterisk prefix on content lines)
  • Preserves all comment content without modification

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated no comments.

Show a summary per file
File Description
Terminal+Input.swift Converts function documentation for readKeyWithTimeout to block style
KittyImage.swift Converts struct-level documentation for KittyImage to block style
ITerm2Image.swift Converts struct-level documentation for ITerm2Image to block style
Terminal.swift Converts function documentation for waitForInput to block style
Logger.swift Converts function documentation for debugLog to block style

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@0xLeif 0xLeif merged commit 8e94788 into main Jan 5, 2026
8 checks passed
@0xLeif 0xLeif deleted the fix/documentation-style branch January 5, 2026 23:44
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