Christopher Hiller

Results 432 comments of Christopher Hiller

Anyhow, if someone from the community knows of a good library, or wants to propose a `util.format`-based solution, that'd be cool. Any changes here should be considered semver-major

@ThisIsMissEm Given that changing the output here would be a breaking change, I'd rather swap out all of it with something better rather than just tweak this one case.

Yeah, I think that test should probably live in `test/node-unit/cli/options.spec.js` instead.

@bcoe If you want to add it to the export map, that'd work for us--we can change our code to use it when we upgrade.

@mf-pherlihy Are you interested in doing further work on this? I'd like to merge the PR, but the test should probably move. If I don't hear back in a couple...

1. Move the test 2. AFAICT you are free to use `applyExtends` as it [has been made part of yargs' public API](https://github.com/yargs/yargs/pull/1733). But it sounds like when we upgrade yargs...

Has something to do with cleanup not happening properly. The fix would be in the runner, reporter, or both.