SBC-Utrecht / PyTomPrivate

Development repository
GNU General Public License v2.0
2 stars 2 forks source link

changed shebang from pytom to python #141

Closed sroet closed 1 year ago

sroet commented 1 year ago

closes SBC-Utrecht/PyTom#50

Updated all occasions I could find with grep -l -r "env pytom" --include=*.py . in the root directory

McHaillet commented 1 year ago

Do the scripts still run as expected? (Except for the license not being printed I assume)

sroet commented 1 year ago

Do the scripts still run as expected? (Except for the license not being printed I assume)

I have run the unittest which complete and don't seem like they give more output than normally

McHaillet commented 1 year ago

I know there are almost no calls to the bin scripts in the unittests. Could you run the e2e-test test_BinScripts.py? That should at least cover some of the bin scripts.

sroet commented 1 year ago

A couple error out but not fail tests. They also error on master (test_localization and test_localizationJob). I will make a separate issue to keep track of this.

McHaillet commented 1 year ago

Okay, good that you made a new issue.

Besides that its good to go for me then.