Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

test: Remove 'src' from pytest test testpaths to allow for non-editable install in CI #1467

Merged
merged 18 commits into from
Oct 19, 2021
Merged

test: Remove 'src' from pytest test testpaths to allow for non-editable install in CI #1467

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
18 commits
Select commit Hold shift + click to select a range
64931e6
remove src from pytest testpaths
matthewfeickert May 18, 2021
3541497
remove --editable from pip install
matthewfeickert May 18, 2021
c4e716d
give doctest-modules a path
matthewfeickert May 18, 2021
8a269e6
Run doctest first
matthewfeickert Oct 13, 2021
0f8d7dd
remove python -m to ensure src dir works
matthewfeickert Oct 13, 2021
b838fec
Remove attempt at path
matthewfeickert Oct 13, 2021
8038eb0
Use pytest only not python -m pytest
matthewfeickert Oct 13, 2021
c820200
use console
matthewfeickert Oct 13, 2021
3f6f4b4
typo
matthewfeickert Oct 13, 2021
a8de0df
Add README examples
matthewfeickert Oct 13, 2021
8215176
Show extra info on skip and fail
matthewfeickert Oct 13, 2021
e164d33
build: Remove in-tree-build pip option
matthewfeickert Oct 13, 2021
4cccfc9
Upload coverage from doctest
matthewfeickert Oct 19, 2021
628105a
Run after doctest coverage upload
matthewfeickert Oct 19, 2021
10be49b
constrain floating point output to not have absurd precision
matthewfeickert Oct 19, 2021
d24139c
Add README into doctests
matthewfeickert Oct 19, 2021
21f1b11
Split addopts across lines
matthewfeickert Oct 19, 2021
e865224
Make testpaths single
matthewfeickert Oct 19, 2021
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
remove --editable from pip install
  • Loading branch information
@matthewfeickert
matthewfeickert committed Oct 19, 2021
commit 35414976955f2402f959fcc18932608270ffcd4b
3 changes: 1 addition & 2 deletions .github/workflows/ci.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -40,8 +40,7 @@ jobs:
- name: Install dependencies
run: |
python -m pip install --upgrade pip setuptools wheel
python -m pip install --upgrade --editable .[test]

python -m pip install --upgrade .[test]
- name: List installed Python packages
run: python -m pip list

Expand Down
8 changes: 4 additions & 4 deletions .github/workflows/dependencies-head.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -28,7 +28,7 @@ jobs:
- name: Install dependencies
run: |
python -m pip install --upgrade pip setuptools wheel
python -m pip --no-cache-dir --quiet install --upgrade --editable .[test]
python -m pip --no-cache-dir --quiet install --upgrade .[test]
python -m pip uninstall --yes scipy
python -m pip install --upgrade --index-url https://pypi.anaconda.org/scipy-wheels-nightly/simple scipy
python -m pip list
Expand All @@ -54,7 +54,7 @@ jobs:
- name: Install dependencies
run: |
python -m pip install --upgrade pip setuptools wheel
python -m pip --no-cache-dir --quiet install --upgrade --editable .[test]
python -m pip --no-cache-dir --quiet install --upgrade .[test]
python -m pip uninstall --yes iminuit
python -m pip install --upgrade cython
python -m pip install --upgrade git+git://github.com/scikit-hep/iminuit.git
Expand All @@ -80,7 +80,7 @@ jobs:
- name: Install dependencies
run: |
python -m pip install --upgrade pip setuptools wheel
python -m pip --no-cache-dir --quiet install --upgrade --editable .[test]
python -m pip --no-cache-dir --quiet install --upgrade .[test]
python -m pip uninstall --yes uproot
python -m pip install --upgrade git+git://github.com/scikit-hep/uproot4.git
python -m pip list
Expand All @@ -105,7 +105,7 @@ jobs:
- name: Install dependencies
run: |
python -m pip install --upgrade pip setuptools wheel
python -m pip --no-cache-dir --quiet install --upgrade --editable .[test]
python -m pip --no-cache-dir --quiet install --upgrade .[test]
python -m pip uninstall --yes pytest
python -m pip install --upgrade git+git://github.com/pytest-dev/pytest.git
python -m pip list
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/lower-bound-requirements.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -28,7 +28,7 @@ jobs:
run: |
python -m pip install --upgrade pip setuptools wheel
python -m pip --no-cache-dir --quiet install --requirement lower-bound-requirements.txt
python -m pip --no-cache-dir --quiet install --editable .[test]
python -m pip --no-cache-dir --quiet install .[test]
python -m pip install --requirement lower-bound-requirements.txt
python -m pip list

Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/notebooks.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,7 @@ jobs:
- name: Install dependencies
run: |
python -m pip install --upgrade pip setuptools wheel
python -m pip --quiet install --upgrade --editable .[complete]
python -m pip --quiet install --upgrade .[complete]
python -m pip list
- name: Test example notebooks
run: |
Expand Down