-
Notifications
You must be signed in to change notification settings - Fork 56
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
pep8-ify function names #32
Comments
@clade's remark from that thread:
|
Maybe for the next major release (v2.0.0), the package name should also be changed to |
That needn't wait - the switch could be made right now, and a reasonably simple |
The dev2 branch is a tentative to pep8-ify the package. The setup.py should install two package : the main one (pydaqmx) and a legacy one (PyDAQmx) that depends on the first one. Changing the name of the all the functions, constant and arguments was not easy so I have decided to completely change the structure of the package and to use object to create the functions. You can see the functions and constants with the commands :
|
The dev2 branch is now tested. For example :
|
Following on from this comment.
So instead of
it would become
This potentially raises the issue that the module name should be PEP8'd as well, becoming the much easier to type
pydaqmx
The text was updated successfully, but these errors were encountered: