ci fixes

Author: HDembinski

.ci/release_check.py

@@ -0,0 +1,29 @@
+"""Ensure that current version is not in conflict with published releases."""
+import requests
+from pkg_resources import parse_version
+import subprocess as subp
+from pathlib import PurePath
+
+project_dir = PurePath(__file__).parent.parent
+version_fn = project_dir / "src/iminuit/version.py"
+changelog_fn = project_dir / "doc/changelog.rst"
+
+with open(version_fn) as f:
+    version = {}
+    exec(f.read(), version)  # this loads __version__
+    version = version["__version__"]
+
+# make sure that changelog was updated
+with open(changelog_fn) as f:
+    assert version in f.read(), "changelog entry missing"
+
+# make sure that version is not already tagged
+tags = subp.check_output(["git", "tag"]).decode().strip().split("\n")
+assert f"v{version}" not in tags, "tag exists"
+
+# make sure that version itself was updated
+r = requests.get("https://pypi.org/pypi/iminuit/json")
+releases = r.json()["releases"]
+pypi_versions = [parse_version(v) for v in releases]
+this_version = parse_version(version)
+assert this_version not in pypi_versions, "pypi version exists"

.ci/version_check.py

@@ -2,9 +2,6 @@ name: Coverage
 
 on:
   pull_request:
-    branches:
-      - master
-      - develop
     paths-ignore:
       - 'doc/**'
       - '.ci/**'
@@ -13,6 +10,7 @@ on:
     branches:
       - master
       - develop
+      - beta/*
 
 jobs:
   coverage:
@@ -29,8 +27,6 @@ jobs:
       run: |
         python -m pip install --upgrade pip wheel
         python -m pip install -r requirements-dev.txt
-    - run: pre-commit run -a
-    # -e is required here to track coverage properly
     - run: python -m pip install -e .
     - run: coverage run -m pytest
     - uses: AndreMiras/coveralls-python-action@develop

.github/workflows/coverage.yml

@@ -2,15 +2,11 @@ name: Docs
 
 on:
   pull_request:
-    branches:
-      - master
-      - develop
     paths-ignore:
       - '.ci/**'
   push:
     branches:
-      - master
-      - develop
+      - beta/*
 
 jobs:
   docs:

.github/workflows/docs.yml

@@ -0,0 +1,22 @@
+name: pre-commit
+
+on:
+  pull_request:
+  push:
+    branches:
+      - master
+      - develop
+      - beta/*
+
+jobs:
+  version:
+    runs-on: ubuntu-latest
+
+    steps:
+    - uses: actions/checkout@v2
+    - uses: actions/setup-python@v2
+      with:
+        python-version: 3.8
+    - run: python -m pip install --upgrade pip wheel
+    - run: python -m pip install pre-commit
+    - run: pre-commit run -a

.github/workflows/pre-commit.yml

@@ -2,9 +2,6 @@ name: Test
 
 on:
   pull_request:
-    # test every branch except master, master is tested by wheels.yml
-    branches-ignore:
-      - master
     paths-ignore:
       - 'doc/**'
       - '.ci/**'

.github/workflows/test.yml

@@ -3,7 +3,7 @@ name: Wheels
 on:
   push:
     branches:
-    - beta-*
+    - beta/*
   pull_request:
     branches:
     - master
@@ -44,7 +44,7 @@ jobs:
           path: ./wheelhouse/*.whl
 
   sdist:
-    name: Build source distribution
+    name: Build source package
     runs-on: ubuntu-latest
     steps:
       - uses: actions/checkout@v2
@@ -68,7 +68,7 @@ jobs:
           path: dist/*.tar.gz
 
   upload:
-    if: "github.event.release.published"
+    if: "github.event.release"
     needs: [wheels, sdist]
     runs-on: ubuntu-latest
     steps:

.github/workflows/version.yml

@@ -32,6 +32,14 @@ repos:
   - id: trailing-whitespace
     exclude: ^doc/_static/.*.svg
 
+- repo: local
+  hooks:
+  - id: version-check
+    name: version-check
+    language: python
+    additional_dependencies: [requests]
+    entry: python .ci/version_check.py
+
 # Python linter (Flake8)
 - repo: https://gitlab.com/pycqa/flake8
   rev: 3.8.4