Skip to content

Conversation

@svandenb-dev
Copy link
Collaborator

This PR is fixing a bug with job manager running on local. When concurrent job were changed many jobs were fired due to a sleep time.
closes #1638
Adding a new argument with the start job manager CLI to select concurrent job.
closes #1639

@svandenb-dev svandenb-dev requested a review from gkorompi November 9, 2025 07:06
@codecov-commenter
Copy link

codecov-commenter commented Nov 9, 2025

Codecov Report

❌ Patch coverage is 3.47222% with 278 lines in your changes missing coverage. Please review.
✅ Project coverage is 70.50%. Comparing base (805037e) to head (4ac06cb).
⚠️ Report is 4 commits behind head on main.

❌ Your patch status has failed because the patch coverage (3.47%) is below the target coverage (85.00%). You can increase the patch coverage or adjust the target coverage.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1640      +/-   ##
==========================================
- Coverage   70.84%   70.50%   -0.34%     
==========================================
  Files         226      227       +1     
  Lines       40690    40924     +234     
==========================================
+ Hits        28826    28855      +29     
- Misses      11864    12069     +205     
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@github-actions github-actions bot added the documentation Improvements or additions to documentation label Nov 10, 2025
gkorompi
gkorompi previously approved these changes Nov 10, 2025
@svandenb-dev svandenb-dev merged commit dde5f86 into main Nov 12, 2025
30 of 31 checks passed
@svandenb-dev svandenb-dev deleted the job-manager-concurrent-job-bug branch November 12, 2025 09:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

FEAT: add concurrent job argument for job manager start CLI BUG: job manager running local fires more jobs

5 participants