Danny Tuppeny
Danny Tuppeny
Removed the unnecessary import, should be good now 🙃 (someone will have to re-approve the bots - thanks!)
@scheglov @devoncarew this landed in the SDK and I've confirmed it's all working as expected :) I don't know how many other projects use test_reflective_loader so publishing this is not...
Does this happen if you use `webdev serve` from the command line? From the error, I don't _think_ this is related to the VS Code extension. I would also check...
Ah, if it's not on `PATH` you would need to run `dart pub global run webdev serve`.
Thanks for testing. I verified I can also reproduce from within VS Code but not from the console. I feel like it's a webdev or dwds issue though. I found...
> merge analyzer_plugins with the main Analyzer isolate. This should cut gigabytes of memory. I'd worry that running more code in the main isolate can only worsen responsiveness in the...
@scheglov this is not a priority, but if you have chance to confirm this change wouldn't negatively affect you, I will work on some tests and changelog/etc.
> We seem to not include `generated` group though. Ah, well spotted, I'll look into that (and ensure there are tests). > With separators it was a tiny bit easier...
> My guess is that we might theoretically have a separate `Reporter` implementation in the analyzer that will join differently. I have some recollection that you can't control the reporter...
It turns out that I only handled `name` for the top level of suites, it was just lost for any nested suite. I've pushed changes to handle this - essentially...