CI has been setup to run the builds all the way to verify phase - which is quality control phase. It is really the minimal phase everybody should run before they make the packaged code available for wider audience. This phase also includes semver checks today. So we should remove the artificial phase binding and let the defaults work.
Bringing the code formatting to the the first phase just makes it impossible to do quick revisions often times done in an environment where IDE is unavailable. Chetan On 11/9/15, 11:19 PM, "Chandni Singh" <chan...@datatorrent.com> wrote: >Because the requirement was to fail the build if any code was added that >breaks a style rule. > >Chandni > >On Mon, Nov 9, 2015 at 11:15 PM, Chetan Narsude <che...@iitbombay.org> >wrote: > >> Chandni - >> >> What's the reason for the check style plugin to be invoked during >>maven's >> validate phase as opposed to verify (the default phase for which the >>plugin >> is written)? >> >> -- >> Chetan >>
