Skip to content

ci: add GitHub Actions workflow for lint and smoke test#310

Merged
Sunrisea merged 1 commit intonacos-group:feature/v2from
cxhello:ci/add-github-actions-v2
Mar 12, 2026
Merged

ci: add GitHub Actions workflow for lint and smoke test#310
Sunrisea merged 1 commit intonacos-group:feature/v2from
cxhello:ci/add-github-actions-v2

Conversation

@cxhello
Copy link

@cxhello cxhello commented Mar 11, 2026

What is the purpose of the change

Closes #305

Add a lightweight CI pipeline to catch code-breaking issues before merge. Based on the direction approved by @Sunrisea in #305, and informed by industry research on similar SDK projects (boto3, dubbo-python, openai-python, nacos-sdk-go).

Brief changelog

ci: add GitHub Actions workflow for lint and smoke test

  • lint job: ruff check with pyflakes rules (F) on v2/ directory, ignoring F401 (unused imports) for now, excluding auto-generated grpcauto/ directory
  • smoke-test job: Python 3.9/3.11/3.13 matrix, install dependencies and verify module imports (import v2.nacos, conditionally import nacos on feature/v2)
  • Both branches (master and feature/v2) use the same workflow file

Note: A companion PR with identical content targets master — see #309.

Add CI pipeline with two jobs:
- lint: ruff check with pyflakes rules (F) on v2/ directory, ignoring
  F401 (unused imports) for now, excluding auto-generated grpcauto directory
- smoke-test: Python 3.9/3.11/3.13 matrix, verify module imports

Closes nacos-group#305

Signed-off-by: cxhello <caixiaohuichn@gmail.com>
@Sunrisea Sunrisea merged commit 9cdd86e into nacos-group:feature/v2 Mar 12, 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