Skip to content

Chore GitHub workflows: refine triggers, align naming, and improve matrices#67

Merged
Borda merged 3 commits intoMiXaiLL76:mainfrom
Lightning-Sandbox:ci/trigers
Jan 23, 2026
Merged

Chore GitHub workflows: refine triggers, align naming, and improve matrices#67
Borda merged 3 commits intoMiXaiLL76:mainfrom
Lightning-Sandbox:ci/trigers

Conversation

@Borda
Copy link
Copy Markdown
Collaborator

@Borda Borda commented Jan 23, 2026

This pull request updates the GitHub Actions workflows to improve CI/CD coverage and maintain consistency in Python version handling. The most important changes are grouped below:

Workflow triggers and coverage:

  • Added push trigger for the main branch to both the documentation build (docs.yml) and test workflows (test.yaml), ensuring these workflows run on direct pushes as well as pull requests. [1] [2]

Python version matrix standardization:

  • Updated the test workflows to use a consistent python-version matrix key instead of pythonv, and expanded the matrix in unittest.yml to test across Python versions 3.9 through 3.13. [1] [2]

Workflow naming:

  • Renamed the workflow in .github/workflows/unittest.yml from "Test" to "Unittests" for clarity.

Copilot AI review requested due to automatic review settings January 23, 2026 03:58
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates GitHub Actions workflows to broaden CI coverage on main pushes, standardize Python matrix naming, and expand unit test version coverage.

Changes:

  • Added push triggers for main to docs.yml and test.yaml.
  • Standardized matrix key to python-version and updated job display names accordingly.
  • Expanded unittest.yml to run against Python 3.9–3.13 with fail-fast: false.

Reviewed changes

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

File Description
.github/workflows/unittest.yml Renames workflow and adds a Python-version matrix for unit tests.
.github/workflows/test.yaml Adds push trigger for main and renames matrix key to python-version.
.github/workflows/docs.yml Adds push trigger for main to ensure docs build runs on direct pushes.

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

Comment thread .github/workflows/unittest.yml Outdated
Comment thread .github/workflows/test.yaml Outdated
Comment thread .github/workflows/test.yaml Outdated
Comment thread .github/workflows/unittest.yml Outdated
Comment thread .github/workflows/unittest.yml Outdated
Borda and others added 2 commits January 23, 2026 09:58
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@MiXaiLL76
Copy link
Copy Markdown
Owner

Hey, friend! Thanks so much for the PR!
lgtm

@Borda Borda merged commit 0eecf5a into MiXaiLL76:main Jan 23, 2026
7 checks passed
@Borda Borda deleted the ci/trigers branch January 23, 2026 10:50
@MiXaiLL76 MiXaiLL76 mentioned this pull request Feb 21, 2026
7 tasks
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.

3 participants