google/closure-compiler

Option to fail on warnings

Open

#3 451 ouverte le 13 août 2019

Voir sur GitHub
 (4 commentaires) (2 réactions) (0 assignés)Java (7 176 stars) (1 197 forks)batch import
help wantedinternal-issue-created

Description

I would like to propose an option that could make the compiler fail if at least one warning is encountered, regardless of errors and exit with a non-zero exit code.

This could be a simple flags such as --fail-on-warnings or a more sophisticated setting like --error-level that could introduce more granular control, i.e. still exit with 0 if compilation failed with errors.

Use case for this is either a purist approach when no warnings are tolerated or code quality control CI job that verifies new code with --checks-only.

Guide contributeur

Stack technique
javajavascript
Domaine
toolingbuild system
Type d'issue
feature
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-2 days
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
stale
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
mostly clear
Prérequis
Javafamiliarity with closure compiler codebase
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
40
Direction de recherche
The issue requests a command line flag to fail on warnings. Look at how existing flags are implemented in the compiler's main entry point, likely in CommandLineRunner.java or similar. Check the warning reporting mechanism and exit code handling. It may require adding an option to the Options class and modifying the compilation process to exit with non zero code when warnings are present.