Skip to content

Fixes fastq mandatory concurrency#14

Open
crystalin wants to merge 1 commit into
tintinweb:masterfrom
crystalin:master
Open

Fixes fastq mandatory concurrency#14
crystalin wants to merge 1 commit into
tintinweb:masterfrom
crystalin:master

Conversation

@crystalin
Copy link
Copy Markdown

fastq complains about concurrency if not explicit:

Error: fastqueue concurrency must be equal to or greater than 1
    at fastqueue (/home/crystalin/.nvm/versions/node/v20.16.0/lib/node_modules/solgrep/node_modules/fastq/queue.js:15:11)
    at /home/crystalin/.nvm/versions/node/v20.16.0/lib/node_modules/solgrep/src/solgrep.js:100:23
    at new Promise (<anonymous>)
    at SolGrep.analyzeDirQueue (/home/crystalin/.nvm/versions/node/v20.16.0/lib/node_modules/solgrep/src/solgrep.js:91:16)
    at main (/home/crystalin/.nvm/versions/node/v20.16.0/lib/node_modules/solgrep/bin/main.js:143:29)
    at Object.<anonymous> (/home/crystalin/.nvm/versions/node/v20.16.0/lib/node_modules/solgrep/bin/main.js:167:1)
    at Module._compile (node:internal/modules/cjs/loader:1358:14)
    at Module._extensions..js (node:internal/modules/cjs/loader:1416:10)
    at Module.load (node:internal/modules/cjs/loader:1208:32)
    at Module._load (node:internal/modules/cjs/loader:1024:12)

Copy link
Copy Markdown

@X1pherW0lf X1pherW0lf left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Note

I encountered the same issue and can confirm that this fix resolves the problem. Thanks for the patch!

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