Skip to content

fix: rebuild CI image when requirements.txt changes#616

Merged
full-bars merged 2 commits into
mainfrom
fix/ci-image-requirements-trigger
Jul 22, 2026
Merged

fix: rebuild CI image when requirements.txt changes#616
full-bars merged 2 commits into
mainfrom
fix/ci-image-requirements-trigger

Conversation

@full-bars

@full-bars full-bars commented Jul 22, 2026

Copy link
Copy Markdown
Owner

Summary

Test plan

Summary by CodeRabbit

  • Chores
    • CI image builds now run automatically when requirements.txt changes.

requirements-dev.txt pulls in requirements.txt via -r, but the path
filter only watched requirements-dev.txt itself, so dependency
additions to requirements.txt (e.g. beautifulsoup4) never triggered
a rebuild of the baked CI test container.
@coderabbitai

coderabbitai Bot commented Jul 22, 2026

Copy link
Copy Markdown

Review Change Stack

📝 Walkthrough

Walkthrough

The CI image build-and-push workflow now runs when requirements.txt changes.

Changes

CI image workflow

Layer / File(s) Summary
Update CI image triggers
.github/workflows/ci-image.yml
Adds requirements.txt to the workflow’s push path filters.

Estimated code review effort: 1 (Trivial) | ~2 minutes

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly matches the main change: rebuilding the CI image when requirements.txt changes.
✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/ci-image-requirements-trigger

Comment @coderabbitai help to get the list of available commands.

@full-bars
full-bars merged commit 131d4d7 into main Jul 22, 2026
10 checks passed
@full-bars
full-bars deleted the fix/ci-image-requirements-trigger branch July 22, 2026 11:36
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