faillint

faillint copied to clipboard

Add ability to lint only test/benchmark/fuzz functions

1

Thank you for this tool! I have a use case where I want to ban `time.Sleep` from test functions to avoid flaky tests. Instead of `time.Sleep` with a hard-coded timeout,...

GiedriusS

I'd like a way to block `github.com/jmoiron/sqlx`, struct `DB`, function `Select`, `Exec`, `Must*` - Must* functions are dangerous in a service context (assuming you don't have `recover` middleware) - Select,...

titpetric

I'd like to forbid dot imports on some projects, could faillint catch package aliases or explicitly `.` with -paths? e.g. ``` import ( . "github.com/pkg/errors" ) ```

titpetric

It would be nice if `-paths` can be parsed from a config file (YAML, JSON, etc). :)

bzon

failint: add ability to lint only tests

2

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...

GiedriusS

Hi, I would like to add your linter to my project. I had my own nix configuration defined, and I thought, why not upstream it. So this is adding the...

angaz

Panic while parsing

1

Seen at https://github.com/grafana/mimir/actions/runs/8520707102/job/23337387670?pr=7356 Of interest, this is a PR to bump Go version from 1.21 to 1.22. Since this repo seems moribund, we will try https://github.com/OpenPeeDeeP/depguard. ``` GOFLAGS="-tags=requires_docker,stringlabels" faillint -paths...

bboreham