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)