Skip to content

Comments

Store failed jobs and add cleanup#126

Merged
adam-fowler merged 7 commits intomainfrom
memory-failed-job
Feb 11, 2026
Merged

Store failed jobs and add cleanup#126
adam-fowler merged 7 commits intomainfrom
memory-failed-job

Conversation

@adam-fowler
Copy link
Member

@adam-fowler adam-fowler commented Feb 10, 2026

Also store paused jobs separately, and reset stopping flag at start up
This is so I can test re-running workflows
Upped default for numWorkers to 16

Also store paused jobs separately, and reset stopping flag at start up
@codecov
Copy link

codecov bot commented Feb 10, 2026

Codecov Report

❌ Patch coverage is 54.76190% with 19 lines in your changes missing coverage. Please review.
✅ Project coverage is 85.54%. Comparing base (6dd9e8e) to head (13ab695).
⚠️ Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
Sources/Jobs/MemoryJobQueue.swift 54.76% 19 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #126      +/-   ##
==========================================
- Coverage   86.67%   85.54%   -1.14%     
==========================================
  Files          25       25              
  Lines        1336     1363      +27     
==========================================
+ Hits         1158     1166       +8     
- Misses        178      197      +19     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@adam-fowler adam-fowler requested a review from Joannis February 10, 2026 14:47
@adam-fowler adam-fowler enabled auto-merge (squash) February 11, 2026 17:33
@adam-fowler adam-fowler merged commit 4d7fbd5 into main Feb 11, 2026
7 of 9 checks passed
@adam-fowler adam-fowler deleted the memory-failed-job branch February 11, 2026 22:11
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.

2 participants