feat: Default to number of processors for exec concurrency

invertase / melos

🌋 A tool for managing Dart projects with multiple packages. With IntelliJ and Vscode IDE support. Supports automated versioning, changelogs & publishing via Conventional Commits.

https://melos.invertase.dev/~melos-latest

Apache License 2.0

1.08k stars 193 forks source link

feat: Default to number of processors for exec concurrency #654

Closed MohiuddinM closed 4 months ago

MohiuddinM commented 4 months ago

Description

This PR removes the hardcoded (5) value of concurrency for exec and replaces it with a dynamic value based on the number of processors in the computer.

Type of Change

[x] ✨ feat -- New feature (non-breaking change which adds functionality)
[ ] 🛠️ fix -- Bug fix (non-breaking change which fixes an issue)
[ ] ❌ ! -- Breaking change (fix or feature that would cause existing functionality to change)
[ ] 🧹 refactor -- Code refactor
[ ] ✅ ci -- Build configuration change
[ ] 📝 docs -- Documentation
[ ] 🗑️ chore -- Chore

CLAassistant commented 4 months ago

All committers have signed the CLA.

MohiuddinM commented 4 months ago

Should I change this here too? https://github.com/invertase/melos/blob/95f23716b33152afb73a1b64a8302138fcbff6f7/packages/melos/lib/src/commands/exec.dart#L21

spydon commented 4 months ago

Should I change this here too?

https://github.com/invertase/melos/blob/95f23716b33152afb73a1b64a8302138fcbff6f7/packages/melos/lib/src/commands/exec.dart#L21

Here you mean? https://github.com/invertase/melos/blob/95f23716b33152afb73a1b64a8302138fcbff6f7/packages/melos/lib/src/commands/exec.dart#L8

Sure, that sounds good!