-
-
Notifications
You must be signed in to change notification settings - Fork 67
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
split documentation #156
split documentation #156
Conversation
Pull Request Test Coverage Report for Build 659902651
💛 - Coveralls |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Very nice and Love the diff!
The documentor code really shows the strength of this package.
Added some possible future improvements as a comment.
{ | ||
return [ | ||
$components = [ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Future improvement is to try to guess all namespaces based on the folder structure (whilst removing the internal once).
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The main issue is with things like Psl\Str\Byte
, Psl\Encoding\Hex
.. etc.
@@ -122,7 +111,7 @@ function get_namespace_documentation(string $namespace): string | |||
} | |||
|
|||
$definition_line = get_symbol_definition_line($symbol, $type); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The definition line for constants always return 0
Maybe we could use better-reflection to guess constants lines in the future.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
maybe later, for now it's not really important.
No description provided.