potiuk commented on code in PR #36537:
URL: https://github.com/apache/airflow/pull/36537#discussion_r1443782845


##########
docs/apache-airflow/extra-packages-ref.rst:
##########
@@ -343,27 +350,278 @@ you need to list explicitly all the non-bundle extras 
that you want to install.
 
+---------------------+-----------------------------------------------------+------------------------------------------------------------------------+
 | all_dbs             | ``pip install 'apache-airflow[all_dbs]'``           | 
All database integrations                                              |
 
+---------------------+-----------------------------------------------------+------------------------------------------------------------------------+
-| devel               | ``pip install 'apache-airflow[devel]'``             | 
Minimum development dependencies (without Hadoop, Kerberos, providers) |
-+---------------------+-----------------------------------------------------+------------------------------------------------------------------------+
-| devel_hadoop        | ``pip install 'apache-airflow[devel_hadoop]'``      | 
Adds Hadoop stack libraries to ``devel`` dependencies                  |
+
+Development extras
+------------------
+
+Generally none of the ``devel`` extras install providers - they expect the 
providers to be used from sources
+and those extras only make sense in editable mode. Users of Airflow should not 
be using them, unless they
+start contributing back and install airflow from sources.
+
+Those extras are only available in Airflow when it is installed in editable 
mode from sources
+(``pip install -e .``).
+
+Devel extras
+============
+
+The devel extras do not install dependencies for features of Airflow, but add 
functionality that is needed to
+develop Airflow, such as running tests, static checks. They do not install 
provider packages - even if they might be related
+to some providers (like ``devel_amazon``) but they might be needed if you want 
to test code of thoe corresponding
+provider.
+
+Even if some ``devel`` extras relate to providers - they do not install 
provider packages - for example
+``devel_amazon`` does not install amazon provider) but they might be needed if 
you want to test code of
+the corresponding provider (for example running mypy checks or running tests).

Review Comment:
   Removed now. yeah. It was repetitive. But with `devel_` removal it's even 
better because both repetitions are gone as we do not need to explain it any 
more... It should **just work** with regular extras.



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

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to