faillint icon indicating copy to clipboard operation
faillint copied to clipboard

failint: add ability to lint only tests

Open GiedriusS opened this issue 2 years ago • 2 comments

Add the ability to lint only testing functions. This is useful to prevent hardcoded sleeps in tests leading to flaky tests.

I have never worked with the AST package in Go so this requires extra carefulness.

Solves https://github.com/fatih/faillint/issues/30.

GiedriusS avatar May 29 '23 11:05 GiedriusS

@fatih kind ping 🤗

GiedriusS avatar Jun 25 '23 09:06 GiedriusS

@fatih ping

GiedriusS avatar Jul 28 '23 09:07 GiedriusS

@GiedriusS thank you. I opened a new PR that does it slightly different. You don't need to traverse the AST, hence it's better that way. Could you try it out? https://github.com/fatih/faillint/pull/43

fatih avatar Apr 03 '24 12:04 fatih

Closing this in favor of https://github.com/fatih/faillint/pull/43

fatih avatar Apr 16 '24 08:04 fatih