-
Notifications
You must be signed in to change notification settings - Fork 96
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
Missing docstrings (Claripy) #17
Comments
I'm interested in doing some work on this issue. Is the list of missing docstrings here up-to-date? |
It is probably pretty close to up to date. The list was probably generated by running an automated tool, so @zardus could provide a way to check for sure. I doubt the situation has changed much though. I would keep in mind that most of these functions aren't user-facing interfaces, so I would prioritizing this by just looking through |
I'm interested in doing some work on this issue. Is there an updated list of missing docstrings? |
@zardus pls post your script for producing this list |
I think something like pylint might be able to generate a warning for each missing or improperly formatted docstring? Then filtering by just that warning might be able to produce a comprehensive list. |
That's exactly what I did. I can't find the original script, but I'll
recreate it!
…On Sun, May 19, 2019 at 5:09 PM Kevin Phoenix ***@***.***> wrote:
I think something like pylint might be able to generate a warning for each
missing or improperly formatted docstring? Then filtering by just that
warning might be able to produce a comprehensive list.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#17?email_source=notifications&email_token=AA2LHFZVEQIWRFESJKPJJWDPWHM3BA5CNFSM4CHA3Z72YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODVXMUWI#issuecomment-493800025>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AA2LHF3X3RX7WHVMLHWXTTDPWHM3BANCNFSM4CHA3Z7Q>
.
|
This issue has been marked as |
This issue has been marked as |
This issue has been closed due to inactivity. |
There are lots of missing module, class, and function docstrings in the angr module. Here is the list. We desperately need help with this from the community, if someone wants to contribute!
Here is the list:
The text was updated successfully, but these errors were encountered: