Re: Coding Style

2014-06-04 Thread Kamil Paral
> > E128 > > > > libtaskotron/check.py:94:17: E128 continuation line under-indented > > for visual indent libtaskotron/check.py:97:21: E128 continuation line > > under-indented for visual indent libtaskotron/check.py:209:21: E128 > > continuation line under-indented for visual indent >

Re: Coding Style

2014-06-04 Thread Kamil Paral
> > But if there's a strong desire for more columns, I'll manage. Can't hinder > > the team, can I? :) > > Also, we should mention that by default the maximal line length is set to 79, > not 80. > > Let's just set it to 80 (as we already use it in the code), and forget about > the heretic 100 ide

Re: Coding Style

2014-06-04 Thread Kamil Paral
> > But if there's a strong desire for more columns, I'll manage. Can't > > hinder the team, can I? :) > > It amuses me that you're strongly "80 columns only" for code but are > pretty anti-column-width-restriction on email :) Yeah, it's a bit ironic. It's mostly about available tools and how the

Re: Depcheck conflicts should be ignored, but are not

2014-06-04 Thread Kamil Paral
> Hello, > > This update: > https://admin.fedoraproject.org/updates/tomcat-native-1.1.30-1.fc20 > ...has some expected conflicts which are flagged by depcheck: > http://autoqa.fedoraproject.org/results/813107-autotest/virt04.qa/depcheck/results/tomcat-native-1.1.30.html > > Now, according to > ht

Depcheck conflicts should be ignored, but are not

2014-06-04 Thread Ville Skyttä
Hello, This update: https://admin.fedoraproject.org/updates/tomcat-native-1.1.30-1.fc20 ...has some expected conflicts which are flagged by depcheck: http://autoqa.fedoraproject.org/results/813107-autotest/virt04.qa/depcheck/results/tomcat-native-1.1.30.html Now, according to https://fedoraprojec