facebook/stetho

Implement checkstyle for travis-CI code style checks

Open

#239 geöffnet am 31. Juli 2015

Auf GitHub ansehen
 (0 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)Java (12.643 Stars) (1.163 Forks)batch import
enhancementhelp wanted

Beschreibung

Now that the number of PRs is starting to heat up, it's time to enforce some of the basic style rules of the project using CI. checkstyle seems like a mature and reasonable way to do that, though I'm open to suggestions :)

Contributor Guide

Tech Stack
java
Domain
ci cdtooling
Issue Type
devops
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 hours
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
stale
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
clear
Voraussetzungen
Java development basicsTravis CI usageCheckstyle configuration
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
50
Research-Richtung
Begin by examining the current repository's CI configuration in the `.travis.yml` file to understand existing build steps. Then, research checkstyle's Maven/Gradle plugin to integrate style checks into the build. Create a `checkstyle.xml` configuration file with the project's coding standards. Finally, update `.travis.yml` to run checkstyle checks after the build.