You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@beam.apache.org by "Ahmet Altay (JIRA)" <ji...@apache.org> on 2016/12/07 01:52:59 UTC

[jira] [Created] (BEAM-1100) Consider removing pylint from mvn verify

Ahmet Altay created BEAM-1100:
---------------------------------

             Summary: Consider removing pylint from mvn verify
                 Key: BEAM-1100
                 URL: https://issues.apache.org/jira/browse/BEAM-1100
             Project: Beam
          Issue Type: Improvement
          Components: sdk-py
            Reporter: Ahmet Altay
            Assignee: Ahmet Altay
            Priority: Minor


pylint depends on git commands for getting a list of changed files to run the linter on. This does not make sense in some cases (for example when in the case code was copied/packaged out of git structure). A few possible remedies could be considered:

- Remove pylint step from mvn verify
- Add an option to pylint run on whole code base and use this in mvn verify
- Change pylint to run on the whole code base (instead of just changed files)



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)