ИМХО, роль линтеров в CI немного не правильно понимается.
Линтеры всё же должны помогать поддерживать качество кода, а не заниматься унижением разработчиков.
То бишь критерий фейла билда должен быть не наличие варнинга в файле / измененной строчке, а в выполнении следующих условий:
* отсутствие критических ошибок в любом месте файла (например, использование не инициализированной переменной)
* отсутствие легко исправимых варнингов, типа неисопльзуемый импорт, не используемый код и т.п.
* число варнингов с уровнем INFO (например Too broad exception clause или Access to a protected member или method can be static) должно быть не больше 10 и не должно увеличиться больше, чем на 1 по сравнению с предыдущем билдом
* То же самое для минорных сообщений, но их может быть чуть больше как во всём файле, так и в изменениях
Ну плюс во время ревью ВСЕ линтеровские варнинги должны подсвечиваться. Пусть ревьютор принимает решение, что это ок или нет.
Если фейлить билд из за любого варнинга, то код будет просто тупо пестрить noinspection. У разработчиков разовьются комплексы и апатия и они по дефолту будут вставлять noinspection не думая.