Fix/requirements into setuppy
Compare changes
Files
5+ 3
− 0
requirements.txt file--ignore=tests/legacy flag to pytest to ignore legacy test dir--cov=cal_tools and --cov=xfel_calibrate to get coverage output for both modules.setup-ssh-euxfel-gitlab template to CIThese changes make the installation of the package a bit more standard, removes the step of having to run two install commands (one for requirements.txt, and one for .), and let us use extras to define additional optional dependencies (e.g. test, dev, etc...).
Tested locally by running pip install without extras, with test, with dev, and with test,dev, then ran the pytests to check that everything was still working correctly.
Also updated the CI definition to use the new setup style, so the tests also executed with the pipeline.
For our local dependencies the name of the package doesn't match the name of the repository, so I have to explicitly write out the name, and then use the git ssh url as a target, e.g. iCalibrationDB @ git+ssh://git@git.xfel.eu:10022/detectors/cal_db_interactive.git@2.0.1. @kluyvert does this look correct?