Parallel execution logic

[lakshayman]

Extract Parallel Execution Logic

Tell me more

What is the issue?

The parallel execution logic is complex and embedded directly in the main script, making it harder to maintain and reuse.

Why this matters

Complex logic embedded in the main script reduces maintainability and makes it difficult to test or modify the parallel execution behavior independently.

Suggested change ∙ Feature Preview

Extract the parallel execution logic into a separate function:

run_parallel_tests() {
    local -r concurrency=$1
    local -a modules=($@)
    local pids=()
    local module_names=()
    
    # ... parallel execution logic ...
}

# Usage
run_parallel_tests $CONCURRENCY "${firebase_modules[@]}"

Provide feedback to improve future suggestions

💬 Looking for more details? Reply to this comment to chat with Korbit.

Originally posted by @korbit-ai[bot] in #245 (comment)