-
-
Notifications
You must be signed in to change notification settings - Fork 18.4k
More clean up of the scripts #19372
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
Comments
|
Beginner Here. Do you want to add this to the Documentation? I can take this up. |
I would also like to help. @datapythonista ? how can we get started on this for you? |
This issue shows to be closed? |
@ProsperousHeart this issue is still open. I think a good start is to add the scripts to the documentation. Feel free to do it the way you think it's better, and send a pull request, so it can be some discussion on your code. |
So just so I understand ... the bullet points are the needs, and the scripts are what the needs are to be applied to? I think that's what is being mentioned here. I'll work on this. ^_^ Thank you! And by "this issue looks to be closed" I was referring to #18949 |
Unable to locate files in 2nd list of docs ... and the 1st list of scripts don't seem to have the issues for docs, though I did find one issue I don't know how to resolve ... author_msg doesn't exist here |
@datapythonista @ProsperousHeart is still an open issue? |
@datapythonista same suggestion w/r/t checkboxes |
Yep, I think it can make sense here. Not sure what's the status of this, but the scripts are now in the linting, so they are probably better than when the issue was created. Will have a look at the state of this next week, and I'll add the checkboxes or I'll create new issues for the pending tasks, so we can move forward with this. |
In #18949, scripts have been improved, to respect PEP-8, to be executable, and a description on what they do has been added to the header. But some tasks are still pending:
%
by new with.format()
Next are the available scripts.
Scripts for the release manager
Script to check the docstrings
Other tools for developers
The text was updated successfully, but these errors were encountered: