-
Notifications
You must be signed in to change notification settings - Fork 13
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
rename DDF.py script to ddfacet #5
Comments
Please explain for those of us who don't know how to see the discussion on slack? This would break any code that calls DDF.py... |
my comment: @o-smirnov: Agreed, let's make it |
If this is to be done -- and it's not my decision -- then someone should at least write a wrapper script DDF.py that prints out a deprecation warning and then calls ddfacet proper. Breaking the user interface in order to comply with one person's style choices is not sensible, whatever your views about whether they're right or not. |
To be fair, it's not as whimsical as all that. When maintaining a repository with dozens of packages, as @gijzelaerr is doing, consistent naming conventions avoid confusion and ultimately save time. There's a reason all Debian packages follow this policy. But we can easily maintain backwards compatibility with a wrapper script or alias -- no sense in punishing the early adopters. :) |
P.S. Same thing applies to naming conventions in the code. But I'm not even thinking of going there.... |
move from pyfits to astropy.io.fits
see discussion on slack.
The text was updated successfully, but these errors were encountered: