Commit c34eba6e authored by Amir MOHAMMADI's avatar Amir MOHAMMADI
Browse files

Merge branch 'pip' into 'master'

Add a check for python -m pip install line

See merge request !252
parents f4cd17bf b9b5ac9d
Pipeline #54688 passed with stages
in 19 minutes and 6 seconds
......@@ -26,7 +26,7 @@ stages:
- python3 bootstrap.py -vv channel base
- source ${CONDA_ROOT}/etc/profile.d/conda.sh
# for conda inspect during builds (on macos)
- conda install -n base conda-build
- mamba install -n base conda-build
- conda activate base
- bdt ci check -vv
cache:
......
......@@ -26,7 +26,7 @@ stages:
- python3 bootstrap.py -vv channel base
- source ${CONDA_ROOT}/etc/profile.d/conda.sh
# for conda inspect during builds (on macos)
- conda install -n base conda-build
- mamba install -n base conda-build
- conda activate base
- bdt ci check -vv
cache:
......
......@@ -1021,6 +1021,18 @@ def check():
f"pyprojecttoml"
)
# check if the package uses pip to install itself
path = "conda/meta.yaml"
if os.path.isfile(path):
with open(path) as f:
content = f.read()
if '- "{{ PYTHON }} -m pip install . -vv"' not in content:
raise RuntimeError(
f"""Could not find the pip install line inside the {path} file.
Please apply similar changes to the {path} file:
https://gitlab.idiap.ch/bob/bob.extension/-/merge_requests/133/diffs"""
)
# if there is a pre-commit configuration file, run the tests
path = ".pre-commit-config.yaml"
if os.path.isfile(path):
......
Supports Markdown
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment