+1 (non-binding) On Fri, Apr 12, 2019 at 8:05 AM Chen Tong <cix...@gmail.com> wrote:
> +1 (non-binding). > > On Fri, Apr 12, 2019 at 9:28 AM Julian De Ruiter < > julianderui...@godatadriven.com> wrote: > > > I also wouldn’t mind including black for checking formatting. > > > > > On 12 Apr 2019, at 15:20, Julian De Ruiter < > > julianderui...@godatadriven.com> wrote: > > > > > > +1 non-binding > > > > > >> On 12 Apr 2019, at 07:25, Maxime Beauchemin < > maximebeauche...@gmail.com> > > wrote: > > >> > > >> +1 (binding) > > >> > > >> Also check out black to auto-pep8! > > >> https://github.com/ambv/black > > >> > > >> On Thu, Apr 11, 2019 at 5:12 PM Tao Feng <fengta...@gmail.com> wrote: > > >> > > >>> +1 > > >>> > > >>> On Thu, Apr 11, 2019 at 4:27 PM Beau Barker < > beauinmelbou...@gmail.com > > > > > >>> wrote: > > >>> > > >>>> +1 non binding > > >>>> > > >>>> Pylint is extremely strict so your may want to be selective about > the > > >>>> rules it applies. > > >>>> > > >>>> > > >>>>> On 12 Apr 2019, at 7:15 am, Kamil Breguła < > kamil.breg...@polidea.com > > > > > >>>> wrote: > > >>>>> > > >>>>> +! (non-binding) > > >>>>> > > >>>>> On Thu, Apr 11, 2019 at 11:05 PM Driesprong, Fokko > > >>> <fo...@driesprong.frl > > >>>>> > > >>>>> wrote: > > >>>>> > > >>>>>> +1 (binding) > > >>>>>> > > >>>>>> For more information about AIP-6: > > >>>>>> > > >>>> > > >>> > > > https://cwiki.apache.org/confluence/pages/viewpage.action?pageId=99844429 > > >>>>>> > > >>>>>> Op do 11 apr. 2019 om 23:03 schreef Kaxil Naik < > kaxiln...@gmail.com > > >: > > >>>>>> > > >>>>>>> +1 (binding) > > >>>>>>> > > >>>>>>> On Thu, Apr 11, 2019 at 9:51 PM Jarek Potiuk < > > >>> jarek.pot...@polidea.com > > >>>>> > > >>>>>>> wrote: > > >>>>>>> > > >>>>>>>> +1 (non-binding). > > >>>>>>>> > > >>>>>>>> On Thu, Apr 11, 2019 at 10:50 PM Bas Harenslak < > > >>>>>>>> basharens...@godatadriven.com> wrote: > > >>>>>>>> > > >>>>>>>>> Hello Airflow community, > > >>>>>>>>> > > >>>>>>>>> This email calls for a vote to introduce Pylint in the Airflow > > >>>>>> project. > > >>>>>>>>> The vote will last for at least 1 week (April 18th 23:00 CET), > > and > > >>> at > > >>>>>>>> least > > >>>>>>>>> three +1 (binding) votes have been cast. > > >>>>>>>>> > > >>>>>>>>> After feedback on AIP-6 and discussion on Slack< > > >>>>>>>>> > > >>>>>> > > https://apache-airflow.slack.com/archives/CCPRP7943/p1554962392081400 > > >>>>>>>> , > > >>>>>>>> I > > >>>>>>>>> propose to vote for adding Pylint<https://pylint.org> to the > > >>> Airflow > > >>>>>>>>> project for static code checking. Pylint complements Flake8 > with > > >>>>>>> stricter > > >>>>>>>>> rules, detects code smells and is customisable so unnecessary > > >>> checks > > >>>>>>> can > > >>>>>>>> be > > >>>>>>>>> ignored. This should benefit the Airflow code base with > > consistent, > > >>>>>>>>> documented code and less errors. > > >>>>>>>>> > > >>>>>>>>> Note that Pylint 2.0 works with Python 3 only, so if the vote > is > > >>>>>>>>> successful, it should be introduced after dropping support for > > >>>>>> Python 2 > > >>>>>>>>> (AIRFLOW-4196< > https://issues.apache.org/jira/browse/AIRFLOW-4196 > > >>>> ). > > >>>>>>>>> Note2: to keep the scope as small as possible, I’ll create a > > >>> separate > > >>>>>>>> vote > > >>>>>>>>> for Black formatting. > > >>>>>>>>> > > >>>>>>>>> Cheers, > > >>>>>>>>> Bas > > >>>>>>>>> > > >>>>>>>> > > >>>>>>>> > > >>>>>>>> -- > > >>>>>>>> > > >>>>>>>> Jarek Potiuk > > >>>>>>>> Polidea <https://www.polidea.com/> | Principal Software > Engineer > > >>>>>>>> > > >>>>>>>> M: +48 660 796 129 <+48660796129> > > >>>>>>>> E: jarek.pot...@polidea.com > > >>>>>>>> > > >>>>>>> > > >>>>>>> > > >>>>>>> -- > > >>>>>>> *Kaxil Naik* > > >>>>>>> *Big Data Consultant *@ *Data Reply UK* > > >>>>>>> *Certified *Google Cloud Data Engineer | *Certified* Apache > Spark & > > >>>> Neo4j > > >>>>>>> Developer > > >>>>>>> *LinkedIn*: https://www.linkedin.com/in/kaxil > > >>>>>>> > > >>>>>> > > >>>>> > > >>>>> > > >>>>> -- > > >>>>> > > >>>>> Kamil Breguła > > >>>>> Polidea <https://www.polidea.com/> | Software Engineer > > >>>>> > > >>>>> M: +48 505 458 451 <+48505458451> > > >>>>> E: kamil.breg...@polidea.com > > >>>>> [image: Polidea] <https://www.polidea.com/> > > >>>>> > > >>>>> We create human & business stories through technology. > > >>>>> Check out our projects! <https://www.polidea.com/our-work> > > >>>>> [image: Github] <https://github.com/Polidea> [image: Facebook] > > >>>>> <https://www.facebook.com/Polidea.Software> [image: Twitter] > > >>>>> <https://twitter.com/polidea> [image: Linkedin] > > >>>>> <https://www.linkedin.com/company/polidea> [image: Instagram] > > >>>>> <https://instagram.com/polidea> [image: Behance] > > >>>>> <https://www.behance.net/polidea> > > >>>> > > >>> > > > > > > > > -- Chao-Han Tsai