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

["ASF GitHub Bot (Jira)" <ji...@apache.org>]

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

ASF GitHub Bot commented on AIRFLOW-6022:
-----------------------------------------

potiuk commented on pull request #6619: [AIRFLOW-6022] Fix CI pylint refresh script 
URL: https://github.com/apache/airflow/pull/6619
 
 
   
 
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


> 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
>
> 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)