Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix return type of getJobCountByTypes #2622

Merged
merged 1 commit into from Jul 5, 2023

Conversation

Benjythebee
Copy link
Contributor

The return type of getJobCountByTypes(types: JobStatus[] | JobStatus) is invalid;
It is a number, not a JobCounts;

Another proof is that function count() uses getJobCountByTypes and yet returns a number;

@manast
Copy link
Member

manast commented Jul 5, 2023

Thanks for the PR. Indeed it was wrong in the typings.

@manast manast merged commit 47722ed into OptimalBits:develop Jul 5, 2023
5 checks passed
@Benjythebee Benjythebee deleted the patch-1 branch July 5, 2023 19:46
github-actions bot pushed a commit that referenced this pull request Aug 8, 2023
# [4.11.0](v4.10.4...v4.11.0) (2023-08-08)

### Bug Fixes

* add mising getMetrics type ([#2640](#2640)) ([a217a7d](a217a7d))
* remove deprecated debuglog ([4ce36fe](4ce36fe))
* **types:** add missing keys to repeat opts ([e4e6457](e4e6457))
* **types:** rename strategyOptions to options to reflect js file ([bae6427](bae6427))
* **typings:** return type of getJobCountByTypes ([#2622](#2622)) ([47722ed](47722ed))
* **worker:** high-memory-usage-when-providing-float-to-concurrency ([#2620](#2620)) ([dcca1e8](dcca1e8))
* change option name to match ts declaration ([909a07e](909a07e))
* ts declaration metrics option and getMetrics function ([11331b7](11331b7))

### Features

* upgrade ioredis to 5.3.2 ([e1883f0](e1883f0))
@manast
Copy link
Member

manast commented Aug 8, 2023

🎉 This PR is included in version 4.11.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

@manast manast added the released label Aug 8, 2023
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.

None yet

2 participants