feat: lint blocking calls

dsherret / dax

Cross-platform shell tools for Deno and Node.js inspired by zx.

MIT License

970 stars 33 forks source link

feat: lint blocking calls #180

Closed sigmaSd closed 7 months ago

sigmaSd commented 10 months ago

Not sure what you think about this but I add this lint a while back and the idea come from dax.

It caught mostly tests which is not that important, but it caught one usage inside cat command, fixing this lint would allow to programmatically stop introducing such things in the codebase again.

sigmaSd commented 10 months ago

Btw on another note I just noticed that the lint name is not in harmony with the others.

seems like it should be sync-fn-inside-async-fn like the other rule that describe the issue not the solution.

sigmaSd commented 7 months ago

Thanks @dsherret maybe if you have time , you can checkout https://github.com/denoland/deno_lint/pull/1199 do you think that the idea there is correct ?