-
-
Notifications
You must be signed in to change notification settings - Fork 1
Add concurrentRunners property to configuration schema for parallel execution #124
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
Conversation
|
|
Co-authored-by: hawkeyexl <[email protected]>
|
Important Review skippedBot user detected. To trigger a single review, invoke the You can disable this status message by setting the 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Join our Discord community for assistance with any issues or questions. Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
Documentation and Community
|
|
📝 Documentation updates detected! New suggestion: Add concurrentRunners configuration property documentation |
This PR adds the
concurrentRunnersproperty to theconfig_v3.schema.jsonto support parallel test execution configuration.Changes Made
Added
concurrentRunnersproperty to the configuration schema with the following validation rules:trueto use CPU core count (capped at 4)falseand invalid values (0, negative numbers, strings, etc.)Added validation examples covering valid usage patterns:
{ "concurrentRunners": 1 }- explicit single runner{ "concurrentRunners": true }- auto-detect CPU cores{ "concurrentRunners": 4 }- multiple runnersSchema Definition
{ "concurrentRunners": { "type": ["integer", "boolean"], "default": 1, "minimum": 1, "description": "Number of concurrent test runners. Set to true to use CPU core count (capped at 4).", "not": { "const": false } } }Validation Results
All acceptance criteria have been verified:
"concurrentRunners": 1"concurrentRunners": true"concurrentRunners": 8(any integer ≥ 1)"concurrentRunners": 0"concurrentRunners": -1"concurrentRunners": false"concurrentRunners": "invalid"The implementation follows existing schema patterns and maintains backward compatibility. All existing tests continue to pass (258 total tests).
Fixes #123.
💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.