allod/tools
2
0
Fork 0
Code Issues 13 Pull requests Projects Releases Packages Wiki Activity Actions

Limit pull-all concurrency #84

Merged
vnprc merged 1 commit from agent/pull-all-limit-jobs into master 2026-07-06 23:40:22 +01:00
Conversation 1 Commits 1 Files changed 3 +104 -4
allod-agent commented 2026-07-06 23:28:19 +01:00
Member
Copy link

Closes #82.

Summary

  • Cap concurrent pull-all repo pulls with PULL_ALL_JOBS, defaulting to 4.
  • Validate PULL_ALL_JOBS as a positive integer while keeping --help usable.
  • Document the tuning knob and add mocked coverage for the concurrency cap.

Risk

R1 low. The behavior change is localized to pull-all; output ordering remains unchanged because results are still collected per repo and printed in workspace order. Rollback is a straight revert.

Validation

  • bash -n workspace/pull-all tests/workspace/pull-all.sh
  • tests/workspace/pull-all.sh
  • tests/workspace/work-diff.sh
  • git diff --check

shellcheck was not installed in this environment.

Closes #82. ## Summary - Cap concurrent `pull-all` repo pulls with `PULL_ALL_JOBS`, defaulting to 4. - Validate `PULL_ALL_JOBS` as a positive integer while keeping `--help` usable. - Document the tuning knob and add mocked coverage for the concurrency cap. ## Risk R1 low. The behavior change is localized to `pull-all`; output ordering remains unchanged because results are still collected per repo and printed in workspace order. Rollback is a straight revert. ## Validation - `bash -n workspace/pull-all tests/workspace/pull-all.sh` - `tests/workspace/pull-all.sh` - `tests/workspace/work-diff.sh` - `git diff --check` `shellcheck` was not installed in this environment.
vnprc approved these changes 2026-07-06 23:40:17 +01:00
vnprc merged commit d1b624e7cd into master 2026-07-06 23:40:22 +01:00
vnprc deleted branch agent/pull-all-limit-jobs 2026-07-06 23:40:22 +01:00
Sign in to join this conversation.
Reviewers
No reviewers
vnprc
Labels
Clear labels
No items
No labels
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
2 participants
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
allod/tools!84
No description provided.