NordicHPC / slurm2sql

Dump slurm accounting database to sqlite3 database for easy analysis
MIT License
9 stars 6 forks source link

Add setup.py #1

Closed rkdarst closed 4 years ago

rkdarst commented 4 years ago
bast commented 4 years ago

Are you planning to share via PyPI at some point?

rkdarst commented 4 years ago

Yes, I want to soon (now, when I have time).

Do you have any comments on the arrangement of setup.py, or how to use PyPI best? I've done it before with envkernel but otherwise don't have much firsthand experience. I know to upload with twine, and beyond that it just worked with a proper setup.py. Anything else to pay attention to?

bast commented 4 years ago

To me the file looked good.

I recommend to try it first on test-PyPI: https://github.com/bast/pypi-howto#upload-to-httpstestpypiorg

But installation from it can give a false failure if the dependencies are not also on test-PyPI.

And before going to test-PyPI I would try to pip install from GitHub or from one folder to another folder but you have probably already done that.

Reason for experimentation: you can remove versions from PyPI but you cannot upload a new payload with the same version so if something does not work, you climb fast that semantic versioning ladder.