Skip to content

feat: update daily caveman optimizer to process 5 files per run#29133

Merged
pelikhan merged 1 commit intomainfrom
copilot/update-daily-cavema-analyzer
Apr 29, 2026
Merged

feat: update daily caveman optimizer to process 5 files per run#29133
pelikhan merged 1 commit intomainfrom
copilot/update-daily-cavema-analyzer

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Apr 29, 2026

Summary

Updates the Daily Caveman Optimizer workflow to process 5 files per run instead of 2.

Changes

  • .github/workflows/daily-caveman-optimizer.md: Changed batch size from 2 to 5 in both:
    • Step 2 (round-robin state): "Pick the next 5 files…"
    • Guidelines section: "Processing 5 files per run…"
  • Recompiled .lock.yml to reflect the updated workflow description.

Copilot AI requested a review from pelikhan April 29, 2026 14:35
@pelikhan pelikhan marked this pull request as ready for review April 29, 2026 14:42
Copilot AI review requested due to automatic review settings April 29, 2026 14:42
@pelikhan pelikhan merged commit 5ffed66 into main Apr 29, 2026
17 of 18 checks passed
@pelikhan pelikhan deleted the copilot/update-daily-cavema-analyzer branch April 29, 2026 14:42
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates the Daily Caveman Optimizer workflow instructions to process a larger batch per run (5 files instead of 2).

Changes:

  • Updated Step 2 round-robin instructions to pick the next 5 files per run.
  • Updated the Guidelines section to reflect 5-file batches.
Show a summary per file
File Description
.github/workflows/daily-caveman-optimizer.md Adjusts the documented batch size and related guidance from 2 to 5 files per run.

Copilot's findings

Tip

Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

  • Files reviewed: 1/1 changed files
  • Comments generated: 2

- **Prefer no change**: When in doubt, leave it alone. The goal is genuine improvement, not churn.
- **One PR per run**: Bundle all changes from the batch into a single PR.
- **Small batches**: Processing 2 files per run keeps each run focused and reviewable.
- **Small batches**: Processing 5 files per run keeps each run focused and reviewable.
Comment on lines 117 to 118
- If the file does not exist, start at index 0.
- If the queue in cache differs from the current file list (files added/removed), rebuild the queue from the current sorted list and reset index to 0.
@github-actions github-actions Bot mentioned this pull request Apr 29, 2026
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.

3 participants