Skip to content

Add support for DocBlock template markers #39

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

Merged
merged 2 commits into from
Sep 2, 2014

Conversation

mvriel
Copy link
Member

@mvriel mvriel commented Aug 30, 2014

DocBlocks may start with #@+ and #@- to indicate that they are (the start) of a DocBlock
template or the end of a template.

In this commit I have changed the way a DocBlock is parsed to interpret this information
and added tests to show for it. In addition I have added more comments to the Regular
Expression responsible for splitting a DocBlock to show the business rules more clearly.

This is the first step in implementing phpDocumentor/phpDocumentor#42.

DocBlocks may start with #@+ and #@- to indicate that they are (the start) of a DocBlock
template or the end of a template.

In this commit I have changed the way a DocBlock is parsed to interpret this information
and added tests to show for it. In addition I have added more comments to the Regular
Expression responsible for splitting a DocBlock to show the business rules more clearly.

This is the first step in implementing phpDocumentor/phpDocumentor#42.
@mvriel
Copy link
Member Author

mvriel commented Aug 30, 2014

@ashnazg can you review this change? There is one broken unit test for the @method tag; that was not caused by this change.

ashnazg added a commit that referenced this pull request Sep 2, 2014
Add support for DocBlock template markers
@ashnazg ashnazg merged commit fd0ac20 into phpDocumentor:master Sep 2, 2014
ashnazg added a commit that referenced this pull request Jun 23, 2015
Add support for DocBlock template markers
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants