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
refactor: type fixes #4326
refactor: type fixes #4326
Conversation
Codecov Report
@@ Coverage Diff @@
## master #4326 +/- ##
==========================================
- Coverage 98.44% 98.44% -0.01%
==========================================
Files 206 206
Lines 7353 7350 -3
Branches 2089 2088 -1
==========================================
- Hits 7239 7236 -3
Misses 55 55
Partials 59 59
Continue to review full report at Codecov.
|
1b8eee5
to
67f74e2
Compare
15484d1
to
6814d5b
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Lots of good stuff!
private indentString: string = undefined as never; | ||
private readonly isEmpty: boolean = true; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
To my understanding it is ok to mark something as readonly if it is only changed in the constructor. It will prevent mutations in all other methods.
src/utils/queue.ts
Outdated
private workerCount = 0; | ||
|
||
constructor(public maxParallel = 1) {} | ||
constructor(private maxParallel = 1) {} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The fallback = 1
is no longer needed (and bad for coverage 😉)
This PR contains:
Are tests included?
Breaking Changes?
List any relevant issue numbers:
Description