amalsaidov opened a new pull request, #76872:
URL: https://github.com/apache/airflow/pull/76872

   Closes #71945.
   
   Improves `airflow-core/docs/core-concepts/operators.rst` (the 
[Operators](https://airflow.apache.org/docs/apache-airflow/stable/core-concepts/operators.html)
 page), addressing the items listed in the issue.
   
   ## What / why
   
   **Typos & capitalization**
   - *Reserved params keyword* section: `In Apache Airflow 2.2.0 ``params`` 
variable is used...` → `In Apache Airflow 2.2.0, the ``params`` variable is 
used...` (adds the missing article and comma), and `third party operators` → 
`third-party operators` (compound adjective).
   - *Difference between pre-/post-execute and setup/teardown* section: `...are 
used to before setup or cleanup operations...` → `...are used to perform setup 
or cleanup operations...` (the sentence was grammatically broken).
   
   **Bug in the pre_execute example**
   - The `_check_skipped` callable built and documented a `tasks_to_skip` 
variable but then tested `if context["task"].task_id in on_ice:`, where 
`on_ice` is undefined. Changed to `tasks_to_skip` so the example is correct and 
consistent with its docstring and `AirflowSkipException` message.
   
   **Pre/post-execute documentation link**
   - Added a `.. seealso::` cross-reference from the pre-/post-execute vs 
setup/teardown section to the setup and teardown how-to 
(`:doc:`/howto/setup-and-teardown``), per the issue's request to link to / 
enhance that documentation. The target already exists and is referenced the 
same way from `core-concepts/dags.rst`.
   
   ## How tested
   - Documentation-only change. Verified every heading underline on the page is 
at least as long as its title (no RST title-underline warnings introduced), the 
`.. seealso::` directive syntax matches other uses in the docs, and the 
`:doc:`/howto/setup-and-teardown`` target file exists.
   - A full Airflow docs build was not run in this environment; the changes are 
small, scoped, and use only patterns already present elsewhere in the docs.
   
   🤖 Generated with [Claude Code](https://claude.com/claude-code)


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