search

spoofax-shell-2017 / spoofax-shell

Apache License 2.0
0 stars 0 forks source link

Add static tooling to pom settings #6

Closed Pathemeous closed 7 years ago

Pathemeous commented 7 years ago

Adds static tooling to the pom settings, including the original checkstyle.xml from the previous group.

Fixes a minor FindBugs violation in Activator.java

Autoformats all files with violations, with checkstyle supressions where auto formatter could not be configured to comply with the settings (example in ReplModule.java where lines did not get wrapped correctly to max 100 characters).

Development branches should rebase on develop/shell after this is merged.

Pathemeous commented 7 years ago

@justinvdk Could you add the javadoc for these methods? Please commit on this branch.

[INFO] --- maven-checkstyle-plugin:2.17:check (verify-style) @ org.metaborg.spoofax.shell.console ---
[WARNING] src/main/java/org/metaborg/spoofax/shell/client/console/strategies/StrategoEvaluationStrategy.java:[14] (javadoc) JavadocType: Missing a Javadoc comment.
[WARNING] src/main/java/org/metaborg/spoofax/shell/client/console/strategies/StrategoEvaluationStrategy.java:[29,5] (javadoc) JavadocMethod: Missing a Javadoc comment.