You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by "Rich Dean (Jira)" <ji...@apache.org> on 2019/11/20 21:31:00 UTC

[jira] [Created] (AIRFLOW-6022) CI pylint script fails first line

Rich Dean created AIRFLOW-6022:
----------------------------------

             Summary: 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


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)