I'm just thinking out laud - perhaps it would be better to add some rules
to checkstyle plugin? This way all commits would have to obey this kind of
Of course there are some drawbacks... we'd need to fix all violations in
existing code base. But this also may be considered as a something good :)
2014-07-30 22:03 GMT+02:00 Anil Saldhana <Anil.Saldhana(a)redhat.com>:
Darran - very good suggestion.
Also, don't merge if there is no javadoc on public methods and classes. :-)
On 07/30/2014 02:46 PM, Darran Lofthouse wrote:
> Just a random idea.
> Can we block merging pull requests if they contain a TODO comment that
> don't reference a Jira issue?
> The views in GitHub are easy to see if a TODO is involved so quite
> simple to double check - and if no Jira is justified maybe the TODO
> isn't either.
> Darran Lofthouse.
> wildfly-dev mailing list
wildfly-dev mailing list