michaelangeloio / does-it-throw

lsp server for finding ES statements that throw!
MIT License
84 stars 0 forks source link

Feature Request: Add @dit-throw to ignore error #105

Closed khuezy closed 8 months ago

khuezy commented 8 months ago

Use case: Some workflows are expected to throw, eg a temporal workflow that aborts on a thrown error.

//@dit-ignore (or equivalent) should tell DIT to ignore the throw statement.

Why: I don't want DIT to give me the squigglies when I expect a function to throw.

michaelangeloio commented 8 months ago

@khuezy do we prefer // @dit-ignore or // @does-it-throw-ignore, or both?

khuezy commented 8 months ago

@michaelangeloio or maybe @throw-ignore? I'm fine with whatever is best.

khuezy commented 8 months ago

@michaelangeloio Can the @ flag be configurable? I kinda like // @it-throws as an answer to does-it-throw.

michaelangeloio commented 8 months ago

118 closes this