You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by GitBox <gi...@apache.org> on 2020/02/21 12:51:29 UTC
[GitHub] [airflow] potiuk commented on a change in pull request #7484:
[AIRFLOW-6864][depends on 6863] Make airflow/jobs pylint compatible
potiuk commented on a change in pull request #7484: [AIRFLOW-6864][depends on 6863] Make airflow/jobs pylint compatible
URL: https://github.com/apache/airflow/pull/7484#discussion_r382564741
##########
File path: airflow/jobs/__init__.py
##########
@@ -16,10 +16,3 @@
# specific language governing permissions and limitations
# under the License.
#
-
-# pylint: disable=missing-docstring
-
-from airflow.jobs.backfill_job import BackfillJob # noqa: F401
-from airflow.jobs.base_job import BaseJob # noqa: F401
-from airflow.jobs.local_task_job import LocalTaskJob # noqa: F401
-from airflow.jobs.scheduler_job import SchedulerJob # noqa: F401
Review comment:
Doubt it. It's really an internal stuff. I see no use of it in Dags (and I think this should be our trigger for UPDATING.md - if you need to change Airlfow configuration or modify your dags or you have different cli/web behaviour.
----------------------------------------------------------------
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
With regards,
Apache Git Services