vaibhav.y added a comment.

@aaron.ballman Would it be possible that I add `::validate` through a follow-up 
PR?

I'm currently checking the JSON output from the writer using Microsoft's online 
validator <https://sarifweb.azurewebsites.net/Validation>, and it is passing. 
Though it tends to complain about things outside of the spec (e.g. the spec 
doesn't constrain the toolComponent version to be numeric but the tool requires 
it to be).


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D109701/new/

https://reviews.llvm.org/D109701

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to