Merge pull request #20 from Aignosi/SIENTIAPDE-1318-adicionar-pipeline-de-qualidade
Enhance Code Quality Validation Script and CI Workflow
This commit is contained in:
31
.github/workflows/quality-gate.yml
vendored
31
.github/workflows/quality-gate.yml
vendored
@@ -57,11 +57,38 @@ jobs:
|
||||
restore-keys: |
|
||||
${{ runner.os }}-pip-
|
||||
|
||||
- name: 📦 Install Dependencies
|
||||
- name: 📦 Install Development Dependencies
|
||||
run: |
|
||||
python -m pip install --upgrade pip
|
||||
pip install -r requirements-dev.txt
|
||||
|
||||
- name: 📦 Install Runtime Dependencies
|
||||
run: |
|
||||
pip install -r ${{ steps.prepare-requirements.outputs.PROCESSED_REQUIREMENTS_FILE }}
|
||||
pip install pytest pytest-cov pytest-asyncio
|
||||
|
||||
- name: 📝 Code Formatting Check (Ruff)
|
||||
run: |
|
||||
echo "Checking code formatting..."
|
||||
ruff format --check scouter/ tests/
|
||||
continue-on-error: false
|
||||
|
||||
- name: 🔎 Code Linting (Ruff)
|
||||
run: |
|
||||
echo "Running linting checks..."
|
||||
ruff check scouter/ tests/
|
||||
continue-on-error: false
|
||||
|
||||
- name: 🏷️ Type Checking (mypy)
|
||||
run: |
|
||||
echo "Running type checks..."
|
||||
mypy scouter/
|
||||
continue-on-error: true
|
||||
|
||||
- name: 🔒 Security Analysis (Bandit)
|
||||
run: |
|
||||
echo "Running security analysis..."
|
||||
bandit -r scouter/ -ll -q
|
||||
continue-on-error: true
|
||||
|
||||
- name: 🧪 Run Tests with Pytest
|
||||
run: |
|
||||
|
||||
29
validate.sh
29
validate.sh
@@ -11,6 +11,27 @@ YELLOW='\033[1;33m'
|
||||
BLUE='\033[0;34m'
|
||||
NC='\033[0m' # No Color
|
||||
|
||||
# Args
|
||||
FIX_MODE=false
|
||||
while [[ $# -gt 0 ]]; do
|
||||
case "$1" in
|
||||
--fix)
|
||||
FIX_MODE=true
|
||||
shift
|
||||
;;
|
||||
-h|--help)
|
||||
echo "Usage: $0 [--fix]"
|
||||
echo " --fix Apply Ruff auto-fixes (format and lint fixes)."
|
||||
exit 0
|
||||
;;
|
||||
*)
|
||||
echo -e "${RED}Unknown option: $1${NC}"
|
||||
echo "Usage: $0 [--fix]"
|
||||
exit 2
|
||||
;;
|
||||
esac
|
||||
done
|
||||
|
||||
echo -e "${BLUE}╔════════════════════════════════════════════════════════╗${NC}"
|
||||
echo -e "${BLUE}║ Model Manager - Code Validation Suite ║${NC}"
|
||||
echo -e "${BLUE}╚════════════════════════════════════════════════════════╝${NC}"
|
||||
@@ -47,12 +68,16 @@ run_step() {
|
||||
FAILED_STEPS=()
|
||||
|
||||
# Step 1: Code Formatting Check (Ruff)
|
||||
if ! run_step "1. Code Formatting (Ruff)" "ruff format scouter/ tests/ && ruff format --check scouter/ tests/"; then
|
||||
# - default: check only
|
||||
# - --fix: write changes
|
||||
if ! run_step "1. Code Formatting (Ruff)" "if \$FIX_MODE; then ruff format scouter/ tests/; else ruff format --check scouter/ tests/; fi"; then
|
||||
FAILED_STEPS+=("Code Formatting")
|
||||
fi
|
||||
|
||||
# Step 2: Linting (Ruff)
|
||||
if ! run_step "2. Code Linting (Ruff)" "ruff check --fix scouter/ tests/"; then
|
||||
# - default: check only
|
||||
# - --fix: apply autofixes
|
||||
if ! run_step "2. Code Linting (Ruff)" "if \$FIX_MODE; then ruff check --fix scouter/ tests/; else ruff check scouter/ tests/; fi"; then
|
||||
FAILED_STEPS+=("Linting")
|
||||
fi
|
||||
|
||||
|
||||
Reference in New Issue
Block a user