ldraney/basketball-api
1
0
Fork 0
Code Issues 90 Pull requests 11 Projects Releases Packages Wiki Activity Actions

fix: Use venv in CI test step to avoid --break-system-packages issue #13

Merged
forgejo_admin merged 1 commit from 12-fix-ci-test-step-fails-due-to-missing-ve into main 2026-03-09 18:29:26 +00:00
Conversation 0 Commits 1 Files changed 1 +5 -4
forgejo_admin commented 2026-03-09 18:26:44 +00:00
Contributor
Copy link

Summary

  • The Woodpecker CI test step fails because pip install .[dev] in a bare python:3.12-slim container is blocked by PEP 668 (externally managed environment)
  • This PR adds a venv so pip installs cleanly without needing --break-system-packages

Changes

  • .woodpecker.yaml: Changed the test step to create a venv at /tmp/venv and run all commands (pip install, ruff check, ruff format, pytest) through the venv's Python/pip. The build-and-push step is unchanged.

Test Plan

  • Woodpecker pipeline passes on this PR (push/pull_request trigger)
  • Verify the build-and-push step still works on merge to main
  • No regressions in CI behavior

Review Checklist

  • Passed automated review-fix loop
  • No secrets committed
  • No unnecessary file changes
  • Commit messages are descriptive

Related

  • Closes #12
  • No plan ancestry -- standalone CI fix
## Summary - The Woodpecker CI test step fails because `pip install .[dev]` in a bare `python:3.12-slim` container is blocked by PEP 668 (externally managed environment) - This PR adds a venv so pip installs cleanly without needing `--break-system-packages` ## Changes - `.woodpecker.yaml`: Changed the test step to create a venv at `/tmp/venv` and run all commands (`pip install`, `ruff check`, `ruff format`, `pytest`) through the venv's Python/pip. The build-and-push step is unchanged. ## Test Plan - [ ] Woodpecker pipeline passes on this PR (push/pull_request trigger) - [ ] Verify the build-and-push step still works on merge to main - [ ] No regressions in CI behavior ## Review Checklist - [ ] Passed automated review-fix loop - [ ] No secrets committed - [ ] No unnecessary file changes - [ ] Commit messages are descriptive ## Related - Closes #12 - No plan ancestry -- standalone CI fix
fix: Use venv in CI test step to avoid --break-system-packages issue
Some checks failed
ci/woodpecker/pr/woodpecker Pipeline failed
Details
2fac5b6739 
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
forgejo_admin deleted branch 12-fix-ci-test-step-fails-due-to-missing-ve 2026-03-09 18:29:26 +00:00
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels   
domain:backend

   
domain:devops

   
domain:frontend

   
status:approved

QA passed, awaiting merge approval

  
status:in-progress

Dev agent is actively working

  
status:needs-fix

QA found issues, back to dev

  
status:qa

PR submitted, awaiting QA review

  
type:bug

Bug fix

  
type:devops

Infrastructure/CI/config work

  
type:feature

New functionality

No labels
domain:backend
domain:devops
domain:frontend
status:approved
status:in-progress
status:needs-fix
status:qa
type:bug
type:devops
type:feature
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
ldraney/basketball-api!13
No description provided.