Skip to content

Commit

Permalink
Give up on not using editable installs in CI
Browse files Browse the repository at this point in the history
I don't know why they're causing issues with loading the Rust extension. But it's not a priority
to figure that out. This PR is still some forward progress.
  • Loading branch information
Eric-Arellano committed Mar 10, 2023
1 parent 73b70b2 commit 68156a1
Show file tree
Hide file tree
Showing 5 changed files with 15 additions and 11 deletions.
4 changes: 2 additions & 2 deletions .azure/lint-linux.yml
Original file line number Diff line number Diff line change
Expand Up @@ -25,10 +25,10 @@ jobs:
-c constraints.txt \
-r requirements.txt \
-r requirements-dev.txt \
-e . # For some reason, we need an editable install for Pylint to find our Rust code.
-e .
displayName: 'Install dependencies'
env:
SETUPTOOLS_ENABLE_FEATURES: "legacy-editable"
SETUPTOOLS_ENABLE_FEATURES: "legacy-editable"
- bash: |
set -e
Expand Down
7 changes: 4 additions & 3 deletions .azure/test-linux.yml
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,6 @@ jobs:
QISKIT_SUPPRESS_PACKAGING_WARNINGS: Y
PIP_CACHE_DIR: $(Pipeline.Workspace)/.pip
QISKIT_TEST_CAPTURE_STREAMS: 1
RUST_DEBUG: 1

steps:
- task: UsePythonVersion@0
Expand Down Expand Up @@ -79,8 +78,10 @@ jobs:
-c constraints.txt \
-r requirements.txt \
-r requirements-dev.txt \
.
-e .
displayName: "Install Terra directly"
env:
SETUPTOOLS_ENABLE_FEATURES: "legacy-editable"
- bash: |
set -e
Expand Down Expand Up @@ -174,7 +175,7 @@ jobs:
image_tests/bin/pip install -U \
-c constraints.txt \
-r requirements.txt \
".[visualization]"
-e ".[visualization]"
sudo apt-get update
sudo apt-get install -y graphviz pandoc
image_tests/bin/pip check
Expand Down
5 changes: 3 additions & 2 deletions .azure/test-macos.yml
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,6 @@ jobs:
QISKIT_SUPPRESS_PACKAGING_WARNINGS: Y
PIP_CACHE_DIR: $(Pipeline.Workspace)/.pip
QISKIT_TEST_CAPTURE_STREAMS: 1
RUST_DEBUG: 1

steps:
- task: UsePythonVersion@0
Expand All @@ -39,9 +38,11 @@ jobs:
-c constraints.txt \
-r requirements.txt \
-r requirements-dev.txt \
.
-e .
pip check
displayName: 'Install dependencies'
env:
SETUPTOOLS_ENABLE_FEATURES: "legacy-editable"
- bash: |
set -e
Expand Down
5 changes: 3 additions & 2 deletions .azure/test-windows.yml
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,6 @@ jobs:
variables:
QISKIT_SUPPRESS_PACKAGING_WARNINGS: Y
QISKIT_TEST_CAPTURE_STREAMS: 1
RUST_DEBUG: 1

steps:
- task: UsePythonVersion@0
Expand Down Expand Up @@ -39,9 +38,11 @@ jobs:
-r requirements.txt \
-r requirements-dev.txt \
"z3-solver" \
.
-e .
pip check
displayName: 'Install dependencies'
env:
SETUPTOOLS_ENABLE_FEATURES: "legacy-editable"
- bash: |
set -e
Expand Down
5 changes: 3 additions & 2 deletions .azure/tutorials-linux.yml
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,6 @@ jobs:
QISKIT_SUPPRESS_PACKAGING_WARNINGS: Y
PIP_CACHE_DIR: $(Pipeline.Workspace)/.pip
QISKIT_CELL_TIMEOUT: 300
RUST_DEBUG: 1

steps:
- task: UsePythonVersion@0
Expand All @@ -36,11 +35,13 @@ jobs:
nbsphinx \
sphinx_rtd_theme \
cvxpy \
.
-e .
sudo apt-get update
sudo apt-get install -y graphviz pandoc
pip check
displayName: 'Install dependencies'
env:
SETUPTOOLS_ENABLE_FEATURES: "legacy-editable"
- bash: |
set -e
Expand Down

0 comments on commit 68156a1

Please sign in to comment.