fix test #122
Workflow file for this run
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
name: Code Quality & Auto-Format Checks | |
# Trigger on push to main/master or PRs targeting these branches | |
on: | |
push: | |
branches: [ main, master ] | |
pull_request: | |
branches: [ main, master ] | |
# Shared environment variables (avoid duplicate hardcoding) | |
env: | |
PYTHON_VERSION: '3.13.7' | |
jobs: | |
# Phase 1: Auto-format with ruff (runs first, controls downstream jobs) | |
ruff-auto-format: | |
name: "π Ruff Auto-Format (With Auto-Commit)" | |
runs-on: ubuntu-latest | |
# Grant write permission for auto-commit (critical for push) | |
permissions: | |
contents: write # Allows workflow to push formatting changes | |
pull-requests: read # Optional: Reads PR info for branch targeting | |
outputs: | |
changes_made: ${{ steps.format-check.outputs.changes_made }} # Track if formatting changes were applied | |
steps: | |
- name: Checkout repository code | |
uses: actions/checkout@v4 | |
with: | |
token: ${{ secrets.GITHUB_TOKEN }} # Uses default token (works with 'contents: write' permission) | |
fetch-depth: 0 # Required for full Git history (needed for commits) | |
ref: ${{ github.head_ref || github.ref }} # Targets PR source branch (avoids merging to main directly) | |
- name: Set up Python ${{ env.PYTHON_VERSION }} | |
uses: actions/setup-python@v4 | |
with: | |
python-version: ${{ env.PYTHON_VERSION }} | |
cache: 'pip' # Cache dependencies to speed up installs | |
- name: Install ruff (code formatter) | |
run: pip install ruff | |
env: | |
PIP_DISABLE_PIP_VERSION_CHECK: 1 # Skip pip version check for faster installs | |
- name: Run ruff format & detect changes | |
id: format-check | |
run: | | |
echo "Running ruff format to fix code styling..." | |
ruff format . # Apply formatting fixes | |
# Check if any files were modified (avoids empty commits) | |
if git diff --quiet --exit-code; then | |
echo "changes_made=false" >> $GITHUB_OUTPUT | |
echo "β No formatting issues found. No commit needed." | |
else | |
echo "changes_made=true" >> $GITHUB_OUTPUT | |
echo "π Formatting changes detected in these files:" | |
git diff --name-only # List modified files for debugging | |
fi | |
- name: Auto-commit & push formatting changes | |
if: steps.format-check.outputs.changes_made == 'true' | |
run: | | |
# Configure Git committer info (required for commits) | |
git config --local user.name "GitHub Actions (Ruff Format)" | |
git config --local user.email "[email protected]" | |
# Commit and push changes | |
git add . | |
git commit -m "[auto] style: Fix code formatting with ruff" # Clear commit message | |
git push | |
echo "β Formatting changes pushed successfully." | |
# Phase 2: Install check tools (runs only after valid ruff-format triggers) | |
setup-check-tools: | |
name: "βοΈ Setup Code Check Tools" | |
needs: ruff-auto-format # Depends on ruff-format completion | |
# Trigger conditions: | |
# - Run on direct pushes to main/master | |
# - Run on PRs only if: 1) ruff made changes, OR 2) PR was merged | |
if: > | |
(github.event_name == 'push') || | |
(github.event_name == 'pull_request' && | |
(needs.ruff-auto-format.outputs.changes_made == 'true' || | |
github.event.pull_request.merged == true)) | |
runs-on: ubuntu-latest | |
steps: | |
- name: Checkout repository code | |
uses: actions/checkout@v4 | |
- name: Set up Python ${{ env.PYTHON_VERSION }} | |
uses: actions/setup-python@v4 | |
with: | |
python-version: ${{ env.PYTHON_VERSION }} | |
cache: 'pip' # Reuse cache from ruff-format job | |
- name: Install all code check tools | |
run: | | |
pip install codespell bandit mypy ruff pytest | |
env: | |
PIP_DISABLE_PIP_VERSION_CHECK: 1 | |
# Non-blocking check: Spell check (fails won't stop workflow) | |
spell-check: | |
name: "π Spell Check (Non-Blocking)" | |
needs: setup-check-tools | |
runs-on: ubuntu-latest | |
steps: | |
- name: Checkout repository code | |
uses: actions/checkout@v4 | |
- name: Set up Python ${{ env.PYTHON_VERSION }} | |
uses: actions/setup-python@v4 | |
with: | |
python-version: ${{ env.PYTHON_VERSION }} | |
cache: 'pip' | |
- name: Run codespell (ignore common false positives) | |
run: | | |
codespell \ | |
--skip="*.json,*.lock,*.csv" \ # Skip non-code files | |
--ignore-words-list="xxx,yyy,zzz" \ # Ignore custom false positives | |
--quiet-level=2 || true # Non-blocking: continue if errors exist | |
# Non-blocking check: Security scan (fails won't stop workflow) | |
security-scan: | |
name: "π Security Scan (Non-Blocking)" | |
needs: setup-check-tools | |
runs-on: ubuntu-latest | |
steps: | |
- name: Checkout repository code | |
uses: actions/checkout@v4 | |
- name: Set up Python ${{ env.PYTHON_VERSION }} | |
uses: actions/setup-python@v4 | |
with: | |
python-version: ${{ env.PYTHON_VERSION }} | |
cache: 'pip' | |
- name: Run bandit (security linter for Python) | |
run: | | |
bandit \ | |
-r . \ # Scan all Python files recursively | |
-f human -o bandit-results.txt \ # Human-readable report | |
-f json -o bandit-results.json || true # JSON report (for tools) + non-blocking | |
# Non-blocking check: Type check (fails won't stop workflow) | |
type-check: | |
name: "π― Type Check (Non-Blocking)" | |
needs: setup-check-tools | |
runs-on: ubuntu-latest | |
steps: | |
- name: Checkout repository code | |
uses: actions/checkout@v4 | |
- name: Set up Python ${{ env.PYTHON_VERSION }} | |
uses: actions/setup-python@v4 | |
with: | |
python-version: ${{ env.PYTHON_VERSION }} | |
cache: 'pip' | |
- name: Run mypy (static type checker) | |
run: | | |
mypy \ | |
--ignore-missing-imports \ # Ignore unresolved imports (e.g., third-party libs) | |
--show-error-codes . || true # Show error codes for debugging + non-blocking | |
# Blocking check: Lint check (fails stop workflow) | |
lint-check: | |
name: "π§Ή Lint Check (Blocking)" | |
needs: setup-check-tools | |
runs-on: ubuntu-latest | |
steps: | |
- name: Checkout repository code | |
uses: actions/checkout@v4 | |
- name: Set up Python ${{ env.PYTHON_VERSION }} | |
uses: actions/setup-python@v4 | |
with: | |
python-version: ${{ env.PYTHON_VERSION }} | |
cache: 'pip' | |
- name: Run ruff check (code linter) | |
run: ruff check --output-format=concise . # Blocking: fails on lint errors | |
# Blocking check: Unit tests (fails stop workflow) | |
unit-tests: | |
name: "π§ͺ Unit Tests (Blocking)" | |
needs: setup-check-tools | |
runs-on: ubuntu-latest | |
steps: | |
- name: Checkout repository code | |
uses: actions/checkout@v4 | |
- name: Set up Python ${{ env.PYTHON_VERSION }} | |
uses: actions/setup-python@v4 | |
with: | |
python-version: ${{ env.PYTHON_VERSION }} | |
cache: 'pip' | |
- name: Run pytest (unit test framework) | |
run: pytest # Blocking: fails on test failures | |
# Security analysis: CodeQL (for vulnerability detection) | |
codeql-analysis: | |
name: "π‘οΈ CodeQL Security Analysis" | |
needs: setup-check-tools # Controlled by ruff-format pre-condition | |
runs-on: ubuntu-latest | |
permissions: | |
actions: read | |
contents: read | |
security-events: write # Required to upload CodeQL results | |
steps: | |
- name: Checkout repository code | |
uses: actions/checkout@v4 | |
- name: Initialize CodeQL | |
uses: github/codeql-action/init@v2 | |
with: | |
languages: python # Analyze Python code | |
- name: Autobuild (auto-configure build for CodeQL) | |
uses: github/codeql-action/autobuild@v2 | |
- name: Run CodeQL analysis | |
uses: github/codeql-action/analyze@v2 | |
with: | |
output: sarif-results/ # Export results for debugging | |
# Final summary: Verify all checks completed | |
all-checks-summary: | |
name: "β All Checks Summary" | |
needs: [spell-check, security-scan, type-check, lint-check, unit-tests, codeql-analysis] | |
if: always() # Run even if some checks fail | |
runs-on: ubuntu-latest | |
steps: | |
- name: Print workflow summary | |
run: | | |
echo "==================== Workflow Summary ====================" | |
echo "Ruff auto-format made changes: ${{ needs.ruff-auto-format.outputs.changes_made }}" | |
echo "---------------------------------------------------------" | |
# Check for blocking failures (lint/tests/CodeQL) | |
if [[ "${{ contains(needs.lint-check.result, 'failure') || contains(needs.unit-tests.result, 'failure') || contains(needs.codeql-analysis.result, 'failure') }}" == "true" ]]; then | |
echo "β Critical failure detected (lint/tests/CodeQL). Fix required." | |
exit 1 # Block workflow on critical failures | |
else | |
echo "β No critical failures. Non-blocking issues (spelling/type) may exist." | |
fi | |