Skip to content

chore: simplify pipx usage and remove redundant example#18

Merged
clouatre merged 1 commit intomainfrom
chore/simplify-pipx-usage
Nov 16, 2025
Merged

chore: simplify pipx usage and remove redundant example#18
clouatre merged 1 commit intomainfrom
chore/simplify-pipx-usage

Conversation

@clouatre
Copy link
Copy Markdown
Member

Summary

Simplifies linting commands by using pre-installed pipx directly, and removes redundant Security Scan example.

Changes

Simplified pipx usage (3 locations)

Before:

run: |
  pipx install uv
  uv tool run ruff check --output-format=json . > lint.json || true

After:

run: pipx run ruff check --output-format=json . > lint.json || true

Updated in:

  • .github/workflows/test-tier1.yml
  • README.md (Quick Start)
  • examples/tier1-maximum-security.yml

Removed redundant example

  • Removed "Security Scan with Artifact Upload" example from README
  • Same pattern as Quick Start, just different AI provider
  • Reduces documentation duplication

Why

  • Faster: No intermediate uv installation needed
  • Simpler: Single line instead of 3
  • Pre-installed: Uses pipx 1.8.0 bundled with Ubuntu 24.04 runners
  • Cleaner docs: Removes redundant example that doesn't add educational value

Testing

  • ✅ Verified pipx 1.8.0 is pre-installed on Ubuntu 24.04 runners
  • ✅ Tested pipx run ruff works locally
  • ✅ No functional changes, only simplification

@clouatre clouatre merged commit 496f3af into main Nov 16, 2025
5 checks passed
@clouatre clouatre deleted the chore/simplify-pipx-usage branch November 16, 2025 19:43
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.

1 participant