mik-laj commented on a change in pull request #10970:
URL: https://github.com/apache/airflow/pull/10970#discussion_r493088944



##########
File path: docs/cli-ref.rst
##########
@@ -35,3 +35,46 @@ development and testing.
    :module: airflow.cli.cli_parser
    :func: get_parser
    :prog: airflow
+
+Environment Variables
+'''''''''''''''''''''
+
+.. envvar:: AIRFLOW__{SECTION}__{KEY}
+
+  Any specific key in a section in Airflow.
+  The user may replace the ``{SECTION}`` placeholder with any section
+  and the ``{KEY}`` placeholder with any key in that specified section.
+  For example, the user may initiate "AIRFLOW__CORE__DAGS_FOLDER",
+  where CORE is the section and DAGS_FOLDER is the key within the section.

Review comment:
       ```suggestion
     For example, If you want to set the ``dags_folder`` options in ``[core]`` 
section, then you 
      should set the ``AIRFLOW__CORE__DAGS_FOLDER``  environment variable.
   ```
   There are a few problems here. First, not the best tone of speech. The word 
may in context suggests that there is another variable name as well. Second, 
all sentences should address the reader directly. Third,  if you refer to a 
section name it should always be written in square brackets as a code literal. 




----------------------------------------------------------------
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:
[email protected]


Reply via email to