-
Notifications
You must be signed in to change notification settings - Fork 295
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
Add requirements to sandbox/ list -- #846
Comments
See http://khmer.readthedocs.org/en/latest/dev/scripts-and-sandbox.html for current; source file is in doc/dev/scripts-and-sandbox.txt. |
@ctb does it mean "only" renaming the scripts in sandbox/ to be all in lowercase? |
I think they are all in lowercase, but they should use '-' instead of '_' as well in the filename. |
working on this |
@mr-c could it be that this can be closed? All sandbox-scripts are in lowercase and use '-' and sandbox/README.rst also looks OK and up to date. Is there anything else to do? |
@ahaerpfer Good catch, thanks! |
directly executable script names should be all lowercase (no CamelCase), and use '-' instead of '_'.
The text was updated successfully, but these errors were encountered: