search

SNEWS2 / SNEWS_Publishing_Tools

Publishing Tool for SNEWS
BSD 3-Clause "New" or "Revised" License
2 stars 2 forks source link

Fix the naming conventions and dependencies #41

Closed KaraMelih closed 2 years ago

KaraMelih commented 2 years ago

There were a several issues that caused faulty installation. Both pip install snews-pt and pip install ./ were working but it was not installing the dependencies due to a package reno and silently bypassing that step. Since we already have all the dependencies in our machines we didn't realize this earlier. However, a clean env installation was not working.

Now all is good. This package I believe is used for dev purposes only. I also moved the requirements.txt to the main folder for some reason otherwise setuptools couldn't read it.

I also changed the name SNEWS_PT -> snews_pt

KaraMelih commented 2 years ago

I tested, also Joe's tests are passing (after the name change). So I'm merging it. Let me know if you notice a mistake / have an objection