[raziel2p]

That's why you make it part of the CI pipeline and require it to pass before merging, making it impossible to forget.

The only reason you'd release a tool like this as a github integration and not an open source library (optionally with a CLI tool) is that you want to get money from it - nothing wrong with that, but that is very much an either/or question.

[rikatee]

Blocking the build works well for things that are boolean "correct" or "not correct", but some things are not are just "food for thought". Django doctor points out when something looks like it can be improved, but that's ultimately up to the dev to take the advice. It's "have you thought of..." rather than "you must do this".

With the advantage that if the dev likes the advice, all they need to do is click "commit"