Kamil Kisiel

Results 109 comments of Kamil Kisiel

Wow, definitely thorough :) So do you think this should just be put alongside the existing scanner package, or should it supersede it?

I don’t believe it does. If you’re interested in adding support a PR with tests would be welcome. > On Dec 6, 2018, at 16:28, Angie Lin wrote: > >...

Honestly I haven't had the time to review it in full or come up with a strategy on migrating the package to a new version.

I'm having trouble getting the switch to work at all, even with filenames like `{Src,Inc}/module.{cpp,h}`

Since this was closed as a duplicate of #7705 I'd just like to add that I think even if the simplified display can be customized, `@warning` should be in the...

I think since we have versioning available in Go I'd be more comfortable adding those to the public API. What do you think @elithrar ?

First step would probably be to get a heap and allocation profile of your application using pprof.

Apparently github.com/molecula/noticeme may implement this. Can look there for details.

Sure, please feel free to submit a PR.

Interesting, I hadn't heard of De Brujin Graphs, I'll have to read up on it. I was thinking of using a radix tree as the data structure for commands, but...