Yoann Padioleau

Results 358 comments of Yoann Padioleau

I think it's fine. We just need to handle the possible out of band errors and report them nicely to the user, without crashing the whole scan

I've commented on the tree-sitter repo

cc @mjambon who knows the best why we use tree-sitter.h instead of api.h

Can we merge this? Did you rebase?

Please remove the UCommon.pr2 and actually add a rule for it in semgrep.jsonnet

@spencerdrak is this still an issue? Closing in doubt.

I see. --test was not designed with multiple target files per rule. But yes maybe we should allow it.

yep, sounds like a useful feature Confidence is not formally specified though right now. It's just part of the metadata. What are the possibilties? low/high/medium?

@amchiclet was talking recently to use Prefab for rollout. (I don't know what it is, but maybe zz has some opinions). I'm closing this PR because of the OSS-pro merge...