You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@airflow.apache.org by "Jimmy Cao (JIRA)" <ji...@apache.org> on 2018/12/21 16:47:00 UTC

[jira] [Created] (AIRFLOW-3555) Remove lxml dependency

Jimmy Cao created AIRFLOW-3555:
----------------------------------

             Summary: Remove lxml dependency
                 Key: AIRFLOW-3555
                 URL: https://issues.apache.org/jira/browse/AIRFLOW-3555
             Project: Apache Airflow
          Issue Type: Bug
          Components: dependencies
    Affects Versions: 1.10.1, 1.10.0, 1.9.0
            Reporter: Jimmy Cao
            Assignee: Jimmy Cao


In this PR: [https://github.com/apache/incubator-airflow/pull/1712/files#diff-948e87b4f8f644b3ad8c7950958df033] lxml was added to airflow/www/views.py, and then in this following PR: [https://github.com/apache/incubator-airflow/pull/1722]  the lxml package was added to the list of core dependencies.

However, months later in this commit: [https://github.com/apache/incubator-airflow/commit/1accb54ff561b8d745277308447dd6f9d3e9f8d5#diff-948e87b4f8f644b3ad8c7950958df033] the lxml import was removed from airflow/www/views.py so it is no longer needed except in the devel extras because it's still used in tests.

It should be removed from the install_requires list.

 



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)