Skip to content

feat: add extractor classes for reflection#24

Open
owenvoke wants to merge 1 commit into
mainfrom
feature/extractor-classes
Open

feat: add extractor classes for reflection#24
owenvoke wants to merge 1 commit into
mainfrom
feature/extractor-classes

Conversation

@owenvoke

Copy link
Copy Markdown
Member

This PR moves logic to 3 "extractor" classes so that it can be reused if needed.

This comment was marked as outdated.

Comment on lines +31 to +34
$comment = $reflection->getDocComment();
$unpadded = PhpDoc::unpad($comment);

return PhpDoc::unwrap($unpadded);

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Do we want to fallback to the PhpDoc comments? 👀

@owenvoke
owenvoke force-pushed the feature/extractor-classes branch 2 times, most recently from 337cc94 to 10b43f1 Compare February 27, 2026 15:35
@owenvoke
owenvoke requested a review from Copilot February 27, 2026 15:36

This comment was marked as outdated.

This comment was marked as outdated.

@owenvoke
owenvoke force-pushed the feature/extractor-classes branch from 269c224 to c498f3f Compare February 27, 2026 16:06

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

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


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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants