airflow-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "ASF subversion and git services (Jira)" <j...@apache.org>
Subject [jira] [Commented] (AIRFLOW-6022) CI pylint script fails first line
Date Thu, 21 Nov 2019 06:42:00 GMT

    [ https://issues.apache.org/jira/browse/AIRFLOW-6022?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16979020#comment-16979020
] 

ASF subversion and git services commented on AIRFLOW-6022:
----------------------------------------------------------

Commit fab957e763f40bf2a2398770312b4834fbd613e1 in airflow's branch refs/heads/master from
Rich Dean
[ https://gitbox.apache.org/repos/asf?p=airflow.git;h=fab957e ]

[AIRFLOW-6022] Move FS defn into awk BEGIN (#6619)



> CI pylint script fails first line
> ---------------------------------
>
>                 Key: AIRFLOW-6022
>                 URL: https://issues.apache.org/jira/browse/AIRFLOW-6022
>             Project: Apache Airflow
>          Issue Type: Bug
>          Components: ci
>    Affects Versions: 1.10.6
>            Reporter: Rich Dean
>            Assignee: Rich Dean
>            Priority: Minor
>             Fix For: 2.0.0
>
>
> The CI scripts use a set of scripts to identify and then hide files which fail pylint
checks. The script which refreshes the list, however, has a bug which causes the first line
to be handled incorrectly.
> The script should take a pylint output line like:
> {code:java}
> airflow/configuration.py:1:0: C0111: Missing function docstring (missing-docstring){code}
> and strip it down to just the filename, like
> {code:java}
> ./airflow/configuration.py{code}
> This is then passed through \{{sort | uniq }} to get a list of unique files. 
> The bug affects the first line of the file, where it doesn't set the field separator
correctly, and instead returns
> {code:java}
> ./airflow/configuration.py:1:0: {code}



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

Mime
View raw message