Skip to content

Update pyproject.toml and CI workflows#29

Merged
MatrixEditor merged 4 commits intoMatrixEditor:masterfrom
StrongWind1:update/base
Mar 14, 2026
Merged

Update pyproject.toml and CI workflows#29
MatrixEditor merged 4 commits intoMatrixEditor:masterfrom
StrongWind1:update/base

Conversation

@StrongWind1
Copy link
Copy Markdown
Contributor

Summary

  • Reorganize pyproject.toml: alphabetize classifiers/dependencies, add keywords, switch [project.optional-dependencies] to [dependency-groups], capitalize URL keys, add Issues link
  • Improve CI workflows: docs deploy only on release (not every push), add verify-ci gate to publish workflow, stricter Sphinx build, better pytest output flags

setuptools-scm requires a [tool.setuptools_scm] section which is
commented out (version is static). License classifier conflicts
with license expression field per PEP 639.
@MatrixEditor MatrixEditor self-requested a review March 14, 2026 07:15
@MatrixEditor MatrixEditor added Dependencies Pull requests / Issues related to dependency updates Type - Enhancement CI/CD Issues and requests related to the CI/CD workflows labels Mar 14, 2026
Comment on lines +21 to 62
verify-ci:
name: Verify required CI workflows
runs-on: ubuntu-latest
timeout-minutes: 10
if: github.event_name == 'release' && github.event.action == 'published'
permissions:
actions: read
contents: read

steps:
- name: Checkout repository
uses: actions/checkout@v6.0.2
with:
fetch-depth: 0
persist-credentials: false

- name: Verify successful CI runs for release commit
env:
GH_TOKEN: ${{ secrets.GITHUB_TOKEN }}
RELEASE_TAG: ${{ github.event.release.tag_name }}
REPOSITORY: ${{ github.repository }}
run: |
set -euo pipefail

release_sha="$(git rev-list -n 1 "refs/tags/${RELEASE_TAG}")"
echo "Checking CI status for ${RELEASE_TAG} at ${release_sha}"

for workflow in ci-lint.yml ci-test.yml; do
conclusion="$(
gh api \
"repos/${REPOSITORY}/actions/workflows/${workflow}/runs?head_sha=${release_sha}&status=completed&event=push&per_page=1" \
--jq '.workflow_runs[0].conclusion // ""'
)"

if [[ "${conclusion}" != "success" ]]; then
echo "Required workflow ${workflow} is not green for ${release_sha}."
echo "Latest completed push-run conclusion: ${conclusion:-<none>}"
exit 1
fi
done

build:
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I would like to add this verification step later on - will be disabled after merging/for the next dev release.

@MatrixEditor MatrixEditor merged commit 013ec2e into MatrixEditor:master Mar 14, 2026
7 checks passed
@StrongWind1 StrongWind1 deleted the update/base branch March 15, 2026 03:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CI/CD Issues and requests related to the CI/CD workflows Dependencies Pull requests / Issues related to dependency updates Type - Enhancement

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants