Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Provide an easy way to enable all of the default warning severity checks #667

Open
davido opened this issue Jul 5, 2017 · 1 comment
Open

Comments

@davido
Copy link
Contributor

davido commented Jul 5, 2017

There is no easy way to enable all of the default warning severity checks. One it's there it could be used for different tool chain integrations, like Bazel's javacopts.

@davido
Copy link
Contributor Author

davido commented Aug 26, 2018

@greggdonovan's team added a workaround for this issue for Bazel toolchain, to enable the individual checks in Bazel: [1]. The interesting question is how can we ensure that this warning list is complete? What if next EP release would add yet another 42 warnings, what would be the appropriate trigger to update this list?

@eaftan @cushon Any thoughts?

I'm asking because I've implemented a different approach for all warnings activation in gerrit: [2], and have now concerns that this list is already outdated. BTW, my implementation was inspired by @cushon's comment: [3].

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant