beam-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Daniel Halperin (JIRA)" <>
Subject [jira] [Updated] (BEAM-1100) Consider removing pylint from mvn verify
Date Fri, 20 Jan 2017 08:00:36 GMT


Daniel Halperin updated BEAM-1100:
    Priority: Critical  (was: Minor)

> Consider removing pylint from mvn verify
> ----------------------------------------
>                 Key: BEAM-1100
>                 URL:
>             Project: Beam
>          Issue Type: Improvement
>          Components: sdk-py
>            Reporter: Ahmet Altay
>            Assignee: Ahmet Altay
>            Priority: Critical
> 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

View raw message