From ade14c67f4bfa96b951506ca34bcbc0e2744694a Mon Sep 17 00:00:00 2001 From: Remis Date: Tue, 1 Sep 2026 20:22:56 +0100 Subject: [PATCH] Install dcio from PyPI dcio 0.1.0 is published, so `pip install -e .` resolves it like any other dependency. Every job used to carry a git install first, because pip could not resolve a requirement that existed nowhere it could reach. install_requires now pins dcio>=0.1.0 rather than naming it with a comment explaining why pip cannot find it. Co-Authored-By: Claude Opus 5 --- .github/workflows/ci.yml | 14 +++----------- setup.py | 5 +---- 2 files changed, 4 insertions(+), 15 deletions(-) diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml index 9802fb5..8bd9e57 100644 --- a/.github/workflows/ci.yml +++ b/.github/workflows/ci.yml @@ -2,8 +2,8 @@ # mattered more once it stopped being self-contained: it now takes its # log-binning from dcio, so a change over there can break it here. # -# dcio is not on PyPI, so every job installs it from git before installing -# EKDIST. That step is the first thing to revisit when dcio is published. +# dcio resolves from PyPI, so `pip install -e .` brings it in like any other +# dependency. It used to need a git install first, which every job carried. name: CI @@ -51,17 +51,10 @@ jobs: with: python-version: ${{ matrix.python }} - # dcio must go in first: it is named in install_requires but cannot be - # resolved from PyPI, so `pip install -e .` would fail without it. - - name: Install dcio from git - shell: bash - run: | - python -m pip install --upgrade pip - pip install git+https://github.com/DCPROGS/dcio@master - - name: Install shell: bash run: | + python -m pip install --upgrade pip pip install -e . pip install pytest pip list @@ -100,7 +93,6 @@ jobs: run: | python -m venv /tmp/clean /tmp/clean/bin/pip install --upgrade pip - /tmp/clean/bin/pip install git+https://github.com/DCPROGS/dcio@master /tmp/clean/bin/pip install . /tmp/clean/bin/pip list diff --git a/setup.py b/setup.py index 4424470..a2e3682 100644 --- a/setup.py +++ b/setup.py @@ -14,10 +14,7 @@ 'numpy', 'scipy', 'matplotlib', - # The shared record layer. Not on PyPI yet, so a source install - # needs it first: - # pip install git+https://github.com/DCPROGS/dcio - 'dcio', + 'dcio>=0.1.0', # the shared record layer #'PyQt5', ], zip_safe=False) \ No newline at end of file