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

(PDOC-272) Add required features attribute #194

Conversation

kris-bosland
Copy link
Contributor

With this change, the required_features argument to newparam and newproperty is parsed.
This data is added to YARD objects and the to_hash output. It is also added to results
for describe, json, and markdown rendering.

@kris-bosland kris-bosland changed the title Pdoc 272 add required features attribute (PDOC-272) add required features attribute Feb 15, 2019
@kris-bosland kris-bosland changed the title (PDOC-272) add required features attribute (PDOC-272) Add required features attribute Feb 15, 2019
@kris-bosland
Copy link
Contributor Author

Edited title - closing and reopening to get JIRA to parse it.

@kris-bosland
Copy link
Contributor Author

@eputnam , @scotje could you review this PR, or suggest someone else?  Thanks!

Copy link
Contributor

@scotje scotje left a comment

Choose a reason for hiding this comment

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

Looks good! Made a note of some extra whitespace in one file.

@@ -99,6 +107,8 @@ def set_values(node, object)
end
end
end


Copy link
Contributor

Choose a reason for hiding this comment

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

can delete this extra whitespace

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Hey @scotje, I think I was having some problem with my commits, but I think it is resolved now. Extra lines are removed!

With this change, the required_features argument to newparam and newproperty is parsed.
This data is added to YARD objects and the to_hash output.  It is also added to results
for describe, json, and markdown rendering.
@kris-bosland kris-bosland force-pushed the pdoc-272-add-required-features-attribute branch from 7d5c2f2 to b65018e Compare March 25, 2019 18:03
@kris-bosland
Copy link
Contributor Author

@scotje , I have cleaned up and done some rebasing - please re-review, thanks!

@scotje scotje merged commit a665166 into puppetlabs:master Apr 3, 2019
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