Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
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
13 changes: 6 additions & 7 deletions .github/actions/install_requirements/action.yml
Original file line number Diff line number Diff line change
Expand Up @@ -29,16 +29,16 @@ runs:

- name: Create lockfile
run: |
mkdir -p lockfiles
pip freeze --exclude-editable > lockfiles/${{ inputs.requirements_file }}
mkdir -p lockfiles-${{ inputs.python_version }}
pip freeze --exclude-editable > lockfiles-${{ inputs.python_version }}/${{ inputs.requirements_file }}
# delete the self referencing line and make sure it isn't blank
sed -i'' -e '/file:/d' lockfiles/${{ inputs.requirements_file }}
sed -i'' -e '/file:/d' lockfiles-${{ inputs.python_version }}/${{ inputs.requirements_file }}
shell: bash

- name: Upload lockfiles
uses: actions/upload-artifact@v3
uses: actions/upload-artifact@v4
with:
name: lockfiles
name: lockfiles-${{ inputs.python_version }}
path: lockfiles
Comment thread
AlexanderWells-diamond marked this conversation as resolved.

# This eliminates the class of problems where the requirements being given no
Expand All @@ -49,10 +49,9 @@ runs:
- name: If requirements file exists, check it matches pip installed packages
run: |
if [ -s ${{ inputs.requirements_file }} ]; then
if ! diff -u ${{ inputs.requirements_file }} lockfiles/${{ inputs.requirements_file }}; then
if ! diff -u ${{ inputs.requirements_file }} lockfiles-${{ inputs.python_version }}/${{ inputs.requirements_file }}; then
echo "Error: ${{ inputs.requirements_file }} need the above changes to be exhaustive"
exit 1
fi
fi
shell: bash

16 changes: 10 additions & 6 deletions .github/workflows/code.yml
Original file line number Diff line number Diff line change
Expand Up @@ -11,17 +11,17 @@ jobs:
fail-fast: false
matrix:
os: ["ubuntu-latest"] # can add windows-latest, macos-latest
python: ["3.8", "3.9", "3.10", "3.11"]
python: ["3.9", "3.10", "3.11", "3.12"]
install: ["-e .[dev]"]
# Make one version be non-editable to test both paths of version code
include:
- os: "ubuntu-latest"
python: "3.7"
python: "3.8"
install: ".[dev]"

runs-on: ${{ matrix.os }}
container:
image: ghcr.io/epics-containers/epics-base-linux-developer:23.6.1
image: ghcr.io/epics-containers/epics-base-developer:7.0.9ec3
volumes:
- /opt/hostedtoolcache:/opt/hostedtoolcache

Expand Down Expand Up @@ -86,9 +86,9 @@ jobs:
CIBW_SKIP: pp* *i686*

- name: Upload sdist and wheels as artifacts
uses: actions/upload-artifact@v3
uses: actions/upload-artifact@v4
with:
name: dist
name: dist-${{ matrix.os }}-${{ matrix.python }}
path: dist
Comment thread
AlexanderWells-diamond marked this conversation as resolved.

- name: Check for packaging errors
Expand All @@ -104,7 +104,11 @@ jobs:
HAS_PYPI_TOKEN: ${{ secrets.PYPI_TOKEN != '' }}

steps:
- uses: actions/download-artifact@v3
- uses: actions/download-artifact@v4
with:
pattern: dist*
path: dist
merge-multiple: true

- name: Fixup blank lockfiles
# Github release artifacts can't be blank
Expand Down
4 changes: 2 additions & 2 deletions pyproject.toml
Original file line number Diff line number Diff line change
Expand Up @@ -16,6 +16,7 @@ classifiers = [
"Programming Language :: Python :: 3.9",
"Programming Language :: Python :: 3.10",
"Programming Language :: Python :: 3.11",
"Programming Language :: Python :: 3.12",
"License :: OSI Approved :: GNU General Public License v2 or later (GPLv2+)",
]
description = "Build tools using cooperative threading"
Expand All @@ -33,7 +34,6 @@ dev = [
"pre-commit",
"pydata-sphinx-theme>=0.12",
"pytest",
"pytest-cov",
"sphinx-autobuild",
"sphinx-copybutton",
"sphinx-design",
Expand Down Expand Up @@ -106,7 +106,7 @@ allowlist_externals =
sphinx-build
sphinx-autobuild
commands =
pytest: pytest --cov=cothread --cov-report term --cov-report xml:cov.xml {posargs}
pytest: pytest {posargs}
pre-commit: pre-commit run --all-files {posargs}
docs: sphinx-{posargs:build -EW --keep-going} -T docs build/html
"""
Loading