Uploaded image for project: 'oVirt - virtualization made easy'
  1. OVIRT-702

add 'isort --check-only' in check-patch.for python projects in addition to pep8 style checking.

    Details

    • Type: By-EMAIL
    • Status: Done
    • Priority: Medium
    • Resolution: Done
    • Affects Version/s: None
    • Fix Version/s: OVIRT-INFRA-NOV-2016
    • Component/s: None
    • Labels:
      None

      Description

      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

        Attachments

          Activity

            People

            • Assignee:
              sbonazzo@redhat.com sbonazzo
              Reporter:
              sbonazzo@redhat.com sbonazzo
            • Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: