fuel-dev team mailing list archive
-
fuel-dev team
-
Mailing list archive
-
Message #00180
Re: Coding guidelines and OpenStack Infra
Andrey,
yes, we have similar tests for it.
But I propose to don't create own task/script/etc for pylint/pep checks. It
would be great to use OpenStack infra for such tasks. It has all necessary
tasks now: checks for pep8, pylint and development docs build for every
review request for every python project. We just need to configure it and
use.
Let's use already created OpenStack infrastructure to be closer to
community!
On Thu, Dec 12, 2013 at 7:33 AM, Andrey Danin <adanin@xxxxxxxxxxxx> wrote:
> I forgot to say the job it triggered by gerrit for every pull request.
>
>
> Sent from my HTC
>
> ----- Reply message -----
> From: "Andrey Danin" <adanin@xxxxxxxxxxxx>
> To: "Ivan Kolodyazhny" <ikolodyazhny@xxxxxxxxxxxx>, "fuel-dev" <
> fuel-dev@xxxxxxxxxxxxxxxxxxx>
> Subject: Re: [Fuel-dev] Coding guidelines and OpenStack Infra
> Date: Thu, Dec 12, 2013 09:31
>
>
> We have a job on fuel-jenkins.mirantis.com which runs flake8 and jslint
> tests and all results are accessible via Internet. Is it not enough?
>
> Sent from my HTC----- Reply message -----
>
> From: "Ivan Kolodyazhny" <ikolodyazhny@xxxxxxxxxxxx>
> To: "fuel-dev" <fuel-dev@xxxxxxxxxxxxxxxxxxx>
> Subject: [Fuel-dev] Coding guidelines and OpenStack Infra
> Date: Wed, Dec 11, 2013 13:59
>
> Hi Fuel developers!
>
> As you know, OpenStack has own coding guidelines
> http://docs.openstack.org/developer/hacking/ and our projects are
> following
> them. But we have no good enough pep8 and pylint checks for every review
> request.
>
> I propose use OpenStack Jenkins to run pep8 and pylint checks using
> OpenStack scripts and settings for them like it works for nova (
> https://jenkins.openstack.org/view/All/job/gate-nova-pep8/,
> https://jenkins.openstack.org/view/All/job/gate-nova-pylint/) and other
> projects.
>
> --
> Regards,
> Ivan "e0ne" Kolodyazhny,
> Software Engineer, Mirantis, Inc.
>
--
Regards,
Ivan Kolodyazhny,
Software Engineer, Mirantis, Inc.
Follow ups
References