Skip to content

fix: added Math.min to the dispatcher and removed redundant checks#152

Merged
GiovaniGuizzo merged 5 commits into
masterfrom
fix/claim-slots
Jan 6, 2026
Merged

fix: added Math.min to the dispatcher and removed redundant checks#152
GiovaniGuizzo merged 5 commits into
masterfrom
fix/claim-slots

Conversation

@GiovaniGuizzo

@GiovaniGuizzo GiovaniGuizzo commented Jan 5, 2026

Copy link
Copy Markdown
Contributor

Checklist for Pull Requests

  • All tests pass (yarn test:all and yarn test:integration)
  • Code follows the style guide and passes lint checks
  • Documentation is updated (README, docs, etc)
  • Linked to corresponding issue, if applicable

Summary of Changes

Fixes a bug where Sidequest would claim more jobs than available in the global pool.
Also removes a snooze of claimed jobs.

Also refactors the workflow to be parallel

@GiovaniGuizzo GiovaniGuizzo merged commit 7fc9e0f into master Jan 6, 2026
6 checks passed
@GiovaniGuizzo GiovaniGuizzo deleted the fix/claim-slots branch January 6, 2026 19:50
sidequest-release Bot pushed a commit that referenced this pull request Jan 6, 2026
## [1.13.3](v1.13.2...v1.13.3) (2026-01-06)

### Bug Fixes

* added Math.min to the dispatcher and removed redundant checks ([#152](#152)) ([7fc9e0f](7fc9e0f))
@sidequest-release

Copy link
Copy Markdown
Contributor

🎉 This PR is included in version 1.13.3 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants