Skip to content

Commit

Permalink
Remove tests for dev pydicom
Browse files Browse the repository at this point in the history
  • Loading branch information
scaramallion committed Aug 11, 2024
1 parent aec6bfa commit 346f73c
Show file tree
Hide file tree
Showing 4 changed files with 95 additions and 96 deletions.
66 changes: 33 additions & 33 deletions .github/workflows/merge-pytest.yml
Original file line number Diff line number Diff line change
Expand Up @@ -64,39 +64,39 @@ jobs:
env:
CODECOV_TOKEN: ${{ secrets.CODECOV_TOKEN }}

pydicom-dev:
# Matrix builds with development pydicom
runs-on: ubuntu-latest
timeout-minutes: 30
strategy:
fail-fast: false
matrix:
python-version: ['3.10', '3.11', '3.12']

steps:
- uses: actions/checkout@v4
with:
fetch-depth: 2
- name: Set up Python ${{ matrix.python-version }}
uses: actions/setup-python@v5
id: setup-python
with:
python-version: ${{ matrix.python-version }}
- name: Install dependencies
run: pip install .[tests]
- name: Switch to development pydicom
run: |
pip install git+https://github.com/pydicom/pydicom
- name: Test with pytest
env:
PYTHON_VERSION: ${{ matrix.python-version }}
run: |
pytest --cov pynetdicom --ignore=pynetdicom/apps && coverage xml
- name: Send coverage results
if: success()
uses: codecov/codecov-action@v4
env:
CODECOV_TOKEN: ${{ secrets.CODECOV_TOKEN }}
# pydicom-dev:
# # Matrix builds with development pydicom
# runs-on: ubuntu-latest
# timeout-minutes: 30
# strategy:
# fail-fast: false
# matrix:
# python-version: ['3.10', '3.11', '3.12']
#
# steps:
# - uses: actions/checkout@v4
# with:
# fetch-depth: 2
# - name: Set up Python ${{ matrix.python-version }}
# uses: actions/setup-python@v5
# id: setup-python
# with:
# python-version: ${{ matrix.python-version }}
# - name: Install dependencies
# run: pip install .[tests]
# - name: Switch to development pydicom
# run: |
# pip install git+https://github.com/pydicom/pydicom
# - name: Test with pytest
# env:
# PYTHON_VERSION: ${{ matrix.python-version }}
# run: |
# pytest --cov pynetdicom --ignore=pynetdicom/apps && coverage xml
# - name: Send coverage results
# if: success()
# uses: codecov/codecov-action@v4
# env:
# CODECOV_TOKEN: ${{ secrets.CODECOV_TOKEN }}

pydicom-release:
# Matrix builds with released pydicom
Expand Down
59 changes: 29 additions & 30 deletions .github/workflows/pr-pytest-apps.yml
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,35 @@ on:

jobs:

pydicom-dev:
# pydicom-dev:
# runs-on: ubuntu-latest
# timeout-minutes: 30
# strategy:
# fail-fast: false
# matrix:
# python-version: ['3.10']
#
# steps:
# - uses: actions/checkout@v4
# with:
# fetch-depth: 2
# - name: Set up Python ${{ matrix.python-version }}
# uses: actions/setup-python@v5
# id: setup-python
# with:
# python-version: ${{ matrix.python-version }}
# - name: Install dependencies
# run: pip install -e .[tests]
# - name: Switch to development pydicom
# run: |
# pip install git+https://github.com/pydicom/pydicom
# - name: Test with pytest
# env:
# PYTHON_VERSION: ${{ matrix.python-version }}
# run: |
# pytest -x --ignore=pynetdicom/tests

pydicom-release:
runs-on: ubuntu-latest
timeout-minutes: 30
strategy:
Expand All @@ -25,37 +53,8 @@ jobs:
python-version: ${{ matrix.python-version }}
- name: Install dependencies
run: pip install -e .[tests]
- name: Switch to development pydicom
run: |
pip install git+https://github.com/pydicom/pydicom
- name: Test with pytest
env:
PYTHON_VERSION: ${{ matrix.python-version }}
run: |
pytest -x --ignore=pynetdicom/tests
# pydicom-release:
# runs-on: ubuntu-latest
# timeout-minutes: 30
# strategy:
# fail-fast: false
# matrix:
# python-version: ['3.9']
#
# steps:
# - uses: actions/checkout@v4
# with:
# fetch-depth: 2
# - name: Set up Python ${{ matrix.python-version }}
# uses: actions/setup-python@v5
# with:
# python-version: ${{ matrix.python-version }}
# - name: Install dependencies
# run: |
# python -m pip install --upgrade pip
# pip install pytest pydicom sqlalchemy pyfakefs
# - name: Test with pytest
# env:
# PYTHON_VERSION: ${{ matrix.python-version }}
# run: |
# pytest --ignore=pynetdicom/tests
62 changes: 31 additions & 31 deletions .github/workflows/pr-pytest.yml
Original file line number Diff line number Diff line change
Expand Up @@ -68,37 +68,37 @@ jobs:
if: ${{ success() }}
uses: codecov/codecov-action@v4

pydicom-dev:
# Matrix builds with development pydicom
runs-on: ubuntu-latest
timeout-minutes: 30
strategy:
fail-fast: false
matrix:
python-version: ['3.10']

steps:
- uses: actions/checkout@v4
with:
fetch-depth: 2
- name: Set up Python ${{ matrix.python-version }}
uses: actions/setup-python@v5
id: setup-python
with:
python-version: ${{ matrix.python-version }}
- name: Install dependencies
run: pip install -e .[tests]
- name: Switch to development pydicom
run: |
pip install git+https://github.com/pydicom/pydicom
- name: Test with pytest
env:
PYTHON_VERSION: ${{ matrix.python-version }}
run: |
pytest -x --cov pynetdicom --ignore=pynetdicom/apps && coverage xml
- name: Send coverage results
if: ${{ success() }}
uses: codecov/codecov-action@v4
# pydicom-dev:
# # Matrix builds with development pydicom
# runs-on: ubuntu-latest
# timeout-minutes: 30
# strategy:
# fail-fast: false
# matrix:
# python-version: ['3.10']
#
# steps:
# - uses: actions/checkout@v4
# with:
# fetch-depth: 2
# - name: Set up Python ${{ matrix.python-version }}
# uses: actions/setup-python@v5
# id: setup-python
# with:
# python-version: ${{ matrix.python-version }}
# - name: Install dependencies
# run: pip install -e .[tests]
# - name: Switch to development pydicom
# run: |
# pip install git+https://github.com/pydicom/pydicom
# - name: Test with pytest
# env:
# PYTHON_VERSION: ${{ matrix.python-version }}
# run: |
# pytest -x --cov pynetdicom --ignore=pynetdicom/apps && coverage xml
# - name: Send coverage results
# if: ${{ success() }}
# uses: codecov/codecov-action@v4

pydicom-release:
# Matrix builds with released pydicom
Expand Down
4 changes: 2 additions & 2 deletions pyproject.toml
Original file line number Diff line number Diff line change
Expand Up @@ -31,7 +31,7 @@ name = "pynetdicom"
readme = "README.rst"
version = "2.2.0.dev0"
requires-python = ">=3.10"
dependencies = ["pydicom >=2.4"]
dependencies = ["pydicom >=2.4, <3"]

[project.urls]
documentation = "https://pydicom.github.io/pynetdicom"
Expand All @@ -48,7 +48,7 @@ dev = [
"pyfakefs >=5.3",
"pytest >=7.4",
"pytest-cov >=4.1",
"ruff >=0.1.5",
"ruff >=0.1",
"sqlalchemy >=2.0",
]
docs = [
Expand Down

0 comments on commit 346f73c

Please sign in to comment.