[JIRA] (OVIRT-702) add 'isort --check-only' in check-patch.for python projects in addition to pep8 style checking.

sbonazzo (oVirt JIRA) jira at ovirt-jira.atlassian.net
Fri Aug 19 12:15:00 UTC 2016


     [ https://ovirt-jira.atlassian.net/browse/OVIRT-702?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

sbonazzo reassigned OVIRT-702:
------------------------------

    Assignee: sbonazzo  (was: infra)

> add 'isort --check-only' in check-patch.for python projects in addition to pep8 style checking.
> -----------------------------------------------------------------------------------------------
>
>                 Key: OVIRT-702
>                 URL: https://ovirt-jira.atlassian.net/browse/OVIRT-702
>             Project: oVirt - virtualization made easy
>          Issue Type: By-EMAIL
>            Reporter: sbonazzo
>            Assignee: sbonazzo
>
> As proposed some time ago, please add 'isort --check-only' in
> check-patch.for python projects in addition to pep8 style checking.
> initial configuration:
> $ cat ~/.isort.cfg [settings] line_length=79
> known_standard_library=configparser,Cheetah.Template
> force_single_line=True default_section=FIRSTPARTY known_otopi=otopi
> known_host_deploy=ovirt_host_deploy known_ovirt_engine=ovirt_engine
> known_ovirt_engine_setup=ovirt_engine_setup
> known_ovirt_setup_lib=ovirt_setup_lib
> known_vdsm=vdsm known_ovirt_hosted_engine_setup=ovirt_hosted_engine_setup
> sections=FUTURE,STDLIB,FIRSTPARTY,OTOPI,VDSM,HOST_DEPLOY,
> OVIRT_ENGINE,OVIRT_ENGINE_SETUP,OVIRT_SETUP_LIB,OVIRT_
> HOSTED_ENGINE_SETUP,THIRDPARTY,LOCALFOLDER lines_between_types=2
> actions:
> 1) update ovirt site on code styling section
> 2) add the check on ovirt-engine project
> 3) add the check to other projects that opts in. As a start all those
> maintained by integration team.
> -- 
> Sandro Bonazzola
> Better technology. Faster innovation. Powered by community collaboration.
> See how it works at redhat.com



--
This message was sent by Atlassian JIRA
(v1000.253.3#100011)



More information about the Infra mailing list