Skip to content

Conversation

@plexoos
Copy link
Member

@plexoos plexoos commented Nov 4, 2025

This change introduces a GitHub Actions workflow to automatically build and test pull requests against all branches. The workflow runs on Ubuntu and validates code quality across multiple Python versions (3.12–3.14).

Key features:
• Runs the test suite on every pull request using a build matrix of Python 3.12–3.14.
• Installs project dependencies and common developer tools (pytest, pytest-cov, mypy, types-requests).
• Performs type checking on the src directory with mypy.
• Executes tests with coverage reporting and posts a summary as a pull request comment (for Python 3.14 job).
• Uses concurrency control to cancel in-progress builds for the same PR.

Additional updates:
• Updated pyproject.toml to silence missing imports for the stomp.* module.
• Added an empty init.py in src/swf_common_lib/ to mark the directory as a package.

Installs mypy and runs static type checks on the src directory during PR builds.
Ensures type consistency is validated in CI.
Adds types-requests to the PR build workflow to resolve missing stubs.
Updates pyproject.toml to silence mypy errors for stomp imports.
@plexoos plexoos force-pushed the add-ci-pr-build branch 7 times, most recently from 496365e to e09bfc6 Compare November 5, 2025 01:33
@BNLNPPS BNLNPPS deleted a comment from github-actions bot Nov 5, 2025
@BNLNPPS BNLNPPS deleted a comment from github-actions bot Nov 5, 2025
@BNLNPPS BNLNPPS deleted a comment from github-actions bot Nov 5, 2025
Extends the pull request build to run pytest with coverage reporting.
@BNLNPPS BNLNPPS deleted a comment from github-actions bot Nov 5, 2025
@BNLNPPS BNLNPPS deleted a comment from github-actions bot Nov 5, 2025
@BNLNPPS BNLNPPS deleted a comment from github-actions bot Nov 5, 2025
@github-actions
Copy link

github-actions bot commented Nov 5, 2025

Test Coverage Summary

Name                                  Stmts   Miss  Cover   Missing
-------------------------------------------------------------------
src/swf_common_lib/__init__.py            0      0   100%
src/swf_common_lib/api_utils.py          33     33     0%   8-80
src/swf_common_lib/base_agent.py        295    295     0%   5-538
src/swf_common_lib/logging_utils.py      33      0   100%
src/swf_common_lib/rest_logging.py       76     76     0%   8-157
-------------------------------------------------------------------
TOTAL                                   437    404     8%
============================== 5 passed in 0.23s ===============================

@plexoos plexoos requested a review from buddhasystem November 5, 2025 16:23
@plexoos plexoos merged commit 4fe8b0e into main Dec 5, 2025
3 checks passed
@plexoos plexoos deleted the add-ci-pr-build branch December 5, 2025 21:39
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.

2 participants