Skip to content

Remove redundant GitHub Actions unit-test job#942

Merged
gladjohn merged 2 commits into
devfrom
copilot/remove-ci-job-from-actions
Jul 10, 2026
Merged

Remove redundant GitHub Actions unit-test job#942
gladjohn merged 2 commits into
devfrom
copilot/remove-ci-job-from-actions

Conversation

Copilot AI commented Jul 10, 2026

Copy link
Copy Markdown
Contributor

The GitHub Actions PR workflow was still running the full Python 3.9–3.14 unit-test matrix even though that coverage now lives in the IDDP/ADO PR pipeline. This trims the workflow down to build/package verification only while keeping PR coverage for sdist/wheel creation and twine check.

  • Workflow scope

    • Renamed .github/workflows/python-package.yml from Build and Unit Tests to Build
    • Updated the header comment to reflect build-only responsibility
    • Clarified that unit tests, E2E, benchmarks, and SDL run in the IDDP/ADO pipeline
  • GitHub Actions job cleanup

    • Removed the ci job entirely
    • Kept the existing build job unchanged:
      • python -m build --sdist --wheel
      • twine check
      • artifact upload
    • Preserved the existing pull_request trigger with no branch filter
  • Related documentation alignment

    • Updated .Pipelines/CI-AND-RELEASE-PIPELINES.md to describe GitHub Actions as PR build verification only
    • Updated doc/python_version_support_policy.md so the supported-Python test matrix now points to the ADO pipeline files rather than the removed GitHub Actions pytest matrix
name: Build

on:
  pull_request:

jobs:
  build:
    name: Build package (sdist + wheel)

Copilot AI requested review from Copilot and removed request for Copilot July 10, 2026 17:25
Copilot AI requested review from Copilot and removed request for Copilot July 10, 2026 17:29
Copilot AI changed the title [WIP] Remove redundant unit testing from GitHub Actions Remove redundant GitHub Actions unit-test job Jul 10, 2026
Copilot AI requested a review from gladjohn July 10, 2026 17:30
@gladjohn gladjohn marked this pull request as ready for review July 10, 2026 17:49
@gladjohn gladjohn requested a review from a team as a code owner July 10, 2026 17:49
Copilot AI review requested due to automatic review settings July 10, 2026 17:49

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the repository’s CI documentation and GitHub Actions PR workflow to reflect that unit/E2E/benchmark/SDL validation now happens in the IDDP/ADO pipelines, leaving GitHub Actions responsible only for PR-time package build and distribution metadata verification.

Changes:

  • Removed the GitHub Actions unit-test matrix job, keeping only python -m build + twine check + artifact upload.
  • Renamed the GitHub Actions workflow from “Build and Unit Tests” to “Build” and updated its header comment to match build-only scope.
  • Updated pipeline/version-support documentation to point Python version test-matrix ownership to ADO pipeline files.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

File Description
doc/python_version_support_policy.md Repoints “supported Python versions encoded in…” references from GitHub Actions to ADO pipeline files.
.Pipelines/CI-AND-RELEASE-PIPELINES.md Updates narrative to describe GitHub Actions as PR package build verification (build + twine check) only.
.github/workflows/python-package.yml Renames workflow and removes the unit-test matrix job, preserving build/package verification steps.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread doc/python_version_support_policy.md
@gladjohn gladjohn merged commit 8ca3c3a into dev Jul 10, 2026
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants