apache / airflow

Apache Airflow - A platform to programmatically author, schedule, and monitor workflows
https://airflow.apache.org/
Apache License 2.0
37.31k stars 14.34k forks source link

Fix description of some extra packages #12042

Closed potiuk closed 4 years ago

potiuk commented 4 years ago

Description

The PR #12023 added consistency check for documentation and setup.py. However few packages need additional description:

This is a great "good-first-issue"

Use case / motivation

Descriptions should be complete.

szn commented 4 years ago

Can you guys help me updating /docs/installation.rst documentation file? All the packages mentioned below are listed there. But the description is missing (and not obvious for me) plus I am not sure if they are listed in a proper section:

  1. @houqp you added pagerduty in d25ef2b2abc3da9f5725a3c433bc035b970eec23
  2. @millertracy you added Plexus in b9dc3c51ba2cba1c61d327488cecf2623d6445b3
  3. @tiopi you added Sentry in dc3bfe470c2d6ad9cda801635e3514088d38aed5
  4. @vsoch you added singularity in 0bb687990b94da7445f4ba081592de8cea73119e
  5. @feluelle you added Tableau in a9ad0a929851b6912e0bb8551f1ff80b50281944
  6. @potiuk you added virtualenv in 7448d3c0cd85770021c230bbea33f758718e300b
vsoch commented 4 years ago

yes of course! Will open a PR soon.

houqp commented 4 years ago

@szn sent a PR to add description for pagerduty provider.

szn commented 4 years ago

@potiuk @feluelle @tiopi @millertracy we can save your time. Instead of creating four separate PR-s (each changing a single line of documentation), you can just write a valid description here and I'll handle the update.

I believe that's faster for you guys.

potiuk commented 4 years ago

Indeed. Good point @szn

szn commented 4 years ago

@potiuk can you generate a description for yandexcloud as well? Is it Yandexcloud hooks and operators?

szn commented 4 years ago

@potiuk can you generate a description for yandexcloud as well? Is it Yandexcloud hooks and operators?

I Googled that, and assumed yes.

Here is PR: https://github.com/apache/airflow/pull/12141