Skip to content
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

Re-parsing #101

Open
wants to merge 6 commits into
base: master
Choose a base branch
from
Open

Re-parsing #101

wants to merge 6 commits into from

Conversation

neild3r
Copy link
Owner

@neild3r neild3r commented Jan 13, 2019

This is the start of a change that will allow you to re-parse an existing docblock adding any missing params etc. This will be tricky because we could be parsing unknown configurations and trying to mix in.

Currently the status is I've got it detecting the range for an existing docblock. I'll move next onto splitting into tags and trying to merge with a new parse

@rmccue
Copy link
Contributor

rmccue commented Aug 24, 2019

This looks great. I'd started work on something similar-ish: adding specific suggestions for @param and @return with more details. Is this something you'd be interested in as well, or do you want to stick to the reparsing technique?

If so, I can send a PR your way. :)

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.

None yet

2 participants