search

open-cli-tools / concurrently

Run commands concurrently. Like `npm run watch-js & npm run watch-less` but better.
https://www.npmjs.com/package/concurrently
MIT License
7k stars 228 forks source link

Delay summary timings output until finished #431

Closed kayahr closed 1 year ago

kayahr commented 1 year ago

This fixes the problem that summary timings are printed before grouped output (Fixes #316).

kayahr commented 1 year ago

Hm... No idea why CI test failed on Windows with Node.js 14 but succeeded on all other platforms including Windows with Node.js 16. Maybe this is a CI environment problem? Huge load on the CI node instance so it ran into a timeout? Can the test be repeated somehow?

coveralls commented 1 year ago

Coverage Status

coverage: 99.391% (+0.001%) from 99.39% when pulling 15311e4037d62857e872ebb547c291b09e42864b on kayahr:feature/log-timings-after-finish into c36f0597ae5b9f497f141039860e9533a2d06a36 on open-cli-tools:main.

paescuj commented 1 year ago

Hm... No idea why CI test failed on Windows with Node.js 14 but succeeded on all other platforms including Windows with Node.js 16. Maybe this is a CI environment problem? Huge load on the CI node instance so it ran into a timeout? Can the test be repeated somehow?

Yeah, this is a timeout we unfortunately run into from time to time... Restarted the failing test 👍