rushstack
rushstack copied to clipboard
[heft] Require a logging name for every operation
Summary
Fixes #4467
Details
@bartvandenende-wm has proposed a simpler fix in PR https://github.com/microsoft/rushstack/pull/4468
However, it seems error-prone and unnecessary to allow operations to be started entirely anonymously. My PR proposes a more aggressive API change to require the name field.
👉 Let's discuss the design in the comments for issue #4467
How it was tested
Redo repro steps from #4467
Impacted documentation
None
any cycles available to cleanup this PR @octogonz? it would be great to see this fix in a future release. I'm happy to help if useful.