Skip to content

Add GitHub Actions CI workflow#2

Merged
shyndman merged 1 commit intomainfrom
copilot/add-github-workflow-for-linting
Apr 27, 2026
Merged

Add GitHub Actions CI workflow#2
shyndman merged 1 commit intomainfrom
copilot/add-github-workflow-for-linting

Conversation

@shyndman
Copy link
Copy Markdown
Owner

This pull request introduces a new GitHub Actions workflow to automate continuous integration (CI) for the repository. The workflow sets up Python, installs dependencies, enforces code quality with linting and formatting checks, performs type checking, and runs unit tests on every pull request and push to the main branch.

New CI workflow automation:

  • Added .github/workflows/ci.yml to define a CI pipeline that triggers on pull requests and pushes to main.

    • Sets up a Python 3.10 environment with dependency caching.
    • Installs project dependencies along with ruff (for linting/formatting) and basedpyright (for type checking).
    • Runs ruff for linting and formatting checks to enforce code quality.
    • Runs basedpyright for type checking, outputs diagnostics, and fails the workflow on type errors.
    • Runs unit tests using Python’s built-in unittest framework.

This addition ensures that code quality and correctness are automatically validated on each change, helping to prevent regressions and maintain standards.

Agent-Logs-Url: https://github.com/shyndman/ptouch-bt/sessions/691de6e5-6fbb-49d7-9880-a42587ceed61

Copilot AI review requested due to automatic review settings April 27, 2026 16:14
Copy link
Copy Markdown

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

Adds a GitHub Actions CI workflow for this Python package to automatically run quality gates and tests on pushes to main and on pull requests.

Changes:

  • Added a CI workflow that sets up Python and installs dependencies/tools.
  • Enforced linting + formatting checks via ruff.
  • Added type checking via basedpyright and test execution via unittest.

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

Comment thread .github/workflows/ci.yml Outdated
Comment thread .github/workflows/ci.yml Outdated
Comment thread .github/workflows/ci.yml Outdated
Comment thread .github/workflows/ci.yml Outdated
Agent-Logs-Url: https://github.com/shyndman/ptouch-bt/sessions/691de6e5-6fbb-49d7-9880-a42587ceed61

Co-authored-by: shyndman <42326+shyndman@users.noreply.github.com>
@shyndman shyndman force-pushed the copilot/add-github-workflow-for-linting branch 2 times, most recently from fa7667f to 9a98c15 Compare April 27, 2026 16:36
@shyndman shyndman merged commit 8004317 into main Apr 27, 2026
1 check passed
@shyndman shyndman deleted the copilot/add-github-workflow-for-linting branch April 27, 2026 16: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.

3 participants