Skip to content

Add GitHub Actions CI for PRs and fix lint/type-check errors - #45

Merged
laixintao merged 4 commits into
masterfrom
copilot/enable-github-workflow
Apr 1, 2026
Merged

Add GitHub Actions CI for PRs and fix lint/type-check errors#45
laixintao merged 4 commits into
masterfrom
copilot/enable-github-workflow

Conversation

Copilot AI commented Apr 1, 2026

Copy link
Copy Markdown
Contributor

CI only ran on master via a broken CircleCI setup (Python 3.7 + tox, no tox.ini). PRs had no CI coverage.

CI workflow

  • Add .github/workflows/ci.yml — matrix CI (Python 3.10/3.11/3.12) on push and PR to main/master
  • Runs ruff, mypy, pytest with coverage via Poetry
  • Remove defunct .circleci/config.yml
  • Add CI status badge to README

Fix pre-existing lint/type errors

  • Fix 3 ruff E501 (line >100) in app.py and summary.py
  • Add src/pingtop/py.typed marker so mypy can check the installed package
  • Add type: ignore[arg-type] for click.style(**kwargs) passthrough in summary.py

Agent-Logs-Url: https://github.com/laixintao/pingtop/sessions/3ab84f5c-8c43-4114-8130-3199dde3b40d

Co-authored-by: laixintao <9675939+laixintao@users.noreply.github.com>
Copilot AI changed the title [WIP] Enable GitHub workflow for repository Add GitHub Actions CI workflow Apr 1, 2026
Copilot AI requested a review from laixintao April 1, 2026 04:36
Copilot stopped work on behalf of laixintao due to an error April 1, 2026 04:38
Copilot stopped work on behalf of laixintao due to an error April 1, 2026 04:38
Copilot AI changed the title Add GitHub Actions CI workflow Add GitHub Actions CI; remove broken CircleCI config Apr 1, 2026
Copilot AI changed the title Add GitHub Actions CI; remove broken CircleCI config Add GitHub Actions CI for PRs and fix lint/type-check errors Apr 1, 2026
Copilot AI requested a review from laixintao April 1, 2026 06:30
@laixintao
laixintao marked this pull request as ready for review April 1, 2026 08:57
@laixintao
laixintao merged commit 318f453 into master Apr 1, 2026
3 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.

2 participants