Make nimble component lint and test run concurrently

[rajsite]

Pull Request

🤨 Rationale

Investigating locally I found that by far the nimble-components lint steps (both eslint and prettier) as well as the karma runs (chrome and firefox) are the slowest to run. Updated the concurrent run so that the component commands run in parallel from the start along with synchronous test and lint runs of every other package.

With this change the following commands all start at the same time:

npm run lint (runs lint sequentially one by one for each package in workspace order)
npm run test (runs test sequentially one by one for each package in workspace order)
npm run -w packages/nimble-components prettier (as part of npm run lint-concurrently)
npm run -w packages/nimble-components lint (as part of npm run lint-concurrently)
npm run -w packages/nimble-components test-firefox:verbose (as part of npm run test-concurrently)
npm run -w packages/nimble-components test-chrome:verbose (as part of npm run test-concurrently)

And can see in the following table that the last to finish are the components commands:

--> ┌───────────────────┬──────────┬───────────┬────────┬───────────────────────────┐
--> │ name              │ duration │ exit code │ killed │ command                   │
--> ├───────────────────┼──────────┼───────────┼────────┼───────────────────────────┤
--> │ lint-concurrently │ 317,863  │ 0         │ false  │ npm run lint-concurrently │
--> │ test-concurrently │ 311,984  │ 0         │ false  │ npm run test-concurrently │
--> │ test              │ 292,637  │ 0         │ false  │ npm run test              │
--> │ lint              │ 269,971  │ 0         │ false  │ npm run lint              │
--> └───────────────────┴──────────┴───────────┴────────┴───────────────────────────┘

[lint-concurrently] --> ┌──────────┬──────────┬───────────┬────────┬──────────────────┐
[lint-concurrently] --> │ name     │ duration │ exit code │ killed │ command          │
[lint-concurrently] --> ├──────────┼──────────┼───────────┼────────┼──────────────────┤
[lint-concurrently] --> │ prettier │ 316,767  │ 0         │ false  │ npm run prettier │
[lint-concurrently] --> │ eslint   │ 93,986   │ 0         │ false  │ npm run eslint   │
[lint-concurrently] --> └──────────┴──────────┴───────────┴────────┴──────────────────┘

[test-concurrently] --> ┌──────────────────────┬──────────┬───────────┬────────┬──────────────────────────────┐
[test-concurrently] --> │ name                 │ duration │ exit code │ killed │ command                      │
[test-concurrently] --> ├──────────────────────┼──────────┼───────────┼────────┼──────────────────────────────┤
[test-concurrently] --> │ test-firefox:verbose │ 310,925  │ 0         │ false  │ npm run test-firefox:verbose │
[test-concurrently] --> │ test-chrome:verbose  │ 303,807  │ 0         │ false  │ npm run test-chrome:verbose  │
[test-concurrently] --> └──────────────────────┴──────────┴───────────┴────────┴──────────────────────────────┘

In the end this ends up shaving off only about 2-3 minutes from the CI. I suspect we are probably at the point of diminishing returns for concurrent execution.

👩‍💻 Implementation

• Renamed the lint and test commands for nimble components so they do not run in the normal lint and test execution for the workspace
• Made the components lint and test commands run their substeps concurrently
• Included the new concurrent component lint and test commands in the top level npm run concurrently-lint-test

🧪 Testing

Tested locally and CI.

✅ Checklist

• [x] I have updated the project documentation to reflect my changes or determined no changes are needed.