Kevin Moore
Kevin Moore
I'd like other eyes on this change first @gmpassos
@TekExplorer – would you give some explicit, but simple code examples of the code you want to support and what you expect this package to generate for you?
i see where you're going here. I'd want to chat with @natebosch or @devoncarew on their thoughts This change should be documented and tested, too!
need changelog entry and tests
> Interesting, there's no runner currently. > > To help me get my head around the command, can you attach the output of `dart run benchmark_harness:bench -h` ? Now in...
> This PR could use a lot more documentation, explaining why things are the way they are. @mosuem – I added a bunch of docs and readme bits. Anything else...
Ideally we'd have a couple of tests to validate the failure cases, but I get that those can be tricky to add.
> @kevmoo What would be the failure cases for this PR? Can you give an example? Unsupported types in the set is the big one. `null` isn't supported, right? Those...
You'll need to rebase this fix.
FYI: I've moved this package to Dart 3.8. You'll need to rebase...