You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Don't worry, no imminent 1.0 release planned 😄 But we've reached a point where we can see some things that we'd want to do but that would potentially break existing integrations, whether via the Go API, CLI or whatnot. I'm mostly starting this issue as a way to compile these things and to gather feedback.
These are just some things that come to mind right now, and I'm sure we'll add more as we think of them:
CLI: The json output format feels largely redundant after sarif was introduced, and I think we should get rid of it in favor of SARIF, as it's a format supported "everywhere". We'd need to ensure there's no data in the current json output that isn't mapped into sarif first though.
Go API: The Location object of a Violation recently got updated to included an optional End position. The starting position col/row are still located directly under Location though. It would be a nice API if these belonged under a Start node.
Rego API: Provide a stable and documented Rego API for writing custom rules. While we've currently documented the process of writing custom rules, it's not clear what rules and helper functions are available to custom rule authors, and we've not really paid much attention to this use case when we rewrite the regal.ast package, for example.
Go API: Try to tighten up the Go API and make public only what's really needed. Once that's done, we can provide a 1.0 "contract". But let's make sure we're not accidentally including things here that aren't generally useful.
The text was updated successfully, but these errors were encountered:
Don't worry, no imminent 1.0 release planned 😄 But we've reached a point where we can see some things that we'd want to do but that would potentially break existing integrations, whether via the Go API, CLI or whatnot. I'm mostly starting this issue as a way to compile these things and to gather feedback.
These are just some things that come to mind right now, and I'm sure we'll add more as we think of them:
json
output format feels largely redundant aftersarif
was introduced, and I think we should get rid of it in favor of SARIF, as it's a format supported "everywhere". We'd need to ensure there's no data in the currentjson
output that isn't mapped intosarif
first though.Location
object of aViolation
recently got updated to included an optionalEnd
position. The starting position col/row are still located directly underLocation
though. It would be a nice API if these belonged under aStart
node.regal.ast
package, for example.The text was updated successfully, but these errors were encountered: