google / google/error-prone

Request: better docs for InjectOnBugCheckers

Open
#3,706 6 comments 0 reactions 0 assignees View on GitHub
Dominant language
Java
Stars
7.2k
Forks
820
Avg merge
5h 9m
Merged PRs (30d)
50

Description

I just got a warning from the new `InjectOnBugCheckers` checker when updating NullAway to EP 2.18.0. But, I'm not sure why the missing `@Inject` annotation on the bug checker constructor is a problem. Could [the docs](https://errorprone.info/bugpattern/InjectOnBugCheckers) be updated to add a bit more explanation? I don't want to have NullAway take another dependence to get the `javax.inject.Inject` annotation, so we'd probably only fix this if it's serious.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.