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
Ruby documentation step 1: placeholders and checks #1683
base: master
Are you sure you want to change the base?
Conversation
…tinue to add docs.
Let's script (or otherwise pre-populate) each of the method's tags/docstrings and get that merged up front. This would:
|
stub-docs.rb
Outdated
|
||
require_relative file | ||
|
||
new_classes = ObjectSpace.each_object(Class).to_a - previous_classes |
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.
Admittedly, running into trouble using this approach of trying to require the file and then get classes/methods out of it. Repeated/nested class/method names cause issues. Instead of getting the script to operate perfectly, it may be a more effective use of time to take over with manual edits.
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.
Go with whatever it easiest. I assume the script is temporary and will be deleted before merging.
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.
Yes, I was only planning to use it as a temporary helper script.
Pull Request Description
Addresses #1682.
Adds cops:
Exclude non "resource" files (and hpxml.rb) from these cops. Stub documentation for all included classes/methods.
Adds a new "Check measures" step to the run-unit-tests GHA job. The step runs
update_measures
and exits if:Add a new test for reporting documentation coverages.
Checklist
PR Author: Check these when they're done. Not all may apply.
strikethroughand check any that do not apply.PR Reviewer: Verify each has been completed.
EPvalidator.xml
) has been updatedopenstudio tasks.rb update_hpxmls
)HPXMLtoOpenStudio/tests/test*.rb
and/orworkflow/tests/test*.rb
)openstudio tasks.rb update_measures
has been run