Copilot commented on code in PR #68655: URL: https://github.com/apache/airflow/pull/68655#discussion_r3425283513
########## contributing-docs/01_roles_in_airflow_project.rst: ########## @@ -55,7 +55,7 @@ The official list of committers can be found `here <https://airflow.apache.org/d Committers are responsible for: -* Championing one or more items on the `Roadmap <https://cwiki.apache.org/confluence/display/AIRFLOW/Airflow+Home>`__ +* Championing one or more items on the `Roadmap <https://cwiki.apache.org/confluence/display/AIRFLOW/Airflow+Wiki>`__ Review Comment: The link text says “Roadmap”, but the target URL points to the generic “Airflow+Wiki” landing page. This is likely misleading for readers; please update the URL to the actual Roadmap page (or rename the link text to match the destination). ########## contributing-docs/01_roles_in_airflow_project.rst: ########## @@ -77,7 +77,7 @@ Contributors are responsible for: * Fixing bugs * Adding features -* Championing one or more items on the `Roadmap <https://cwiki.apache.org/confluence/display/AIRFLOW/Airflow+Home>`__. +* Championing one or more items on the `Roadmap <https://cwiki.apache.org/confluence/display/AIRFLOW/Airflow+Wiki>`__. Review Comment: The link text says “Roadmap”, but the target URL points to the generic “Airflow+Wiki” landing page. This is likely misleading for readers; please update the URL to the actual Roadmap page (or rename the link text to match the destination). ########## contributing-docs/02_how_to_communicate.rst: ########## @@ -49,7 +49,7 @@ You can join the channels via links at the `Airflow Community page <https://airf * general issues, asking community for opinion * discussing proposals * voting -* The `Airflow CWiki <https://cwiki.apache.org/confluence/display/AIRFLOW/Airflow+Home?src=breadcrumbs>`_ for: +* The `Airflow CWiki <https://cwiki.apache.org/confluence/display/AIRFLOW/Airflow+Wiki?src=breadcrumbs>`_ for: Review Comment: The CWiki URL includes a UI-specific query parameter (`?src=breadcrumbs`). Prefer using the canonical URL without the query string to reduce churn and avoid brittle links. ########## contributing-docs/01_roles_in_airflow_project.rst: ########## @@ -64,7 +64,7 @@ Release managers ---------------- The task of release managers is to prepare and release Airflow artifacts (airflow, providers, Helm Chart, Python client). -The release managers are usually PMC members and the process of releasing is described in the `dev <dev>`__ +The release managers are usually PMC members and the process of releasing is described in the `dev <../dev>`__ Review Comment: The `../dev` target may not resolve correctly in the rendered docs if it points to a directory (or if the docs builder doesn’t preserve this path structure). Consider linking to a specific page (e.g., an index/README document) or using the appropriate Sphinx role (like `:doc:`) if this documentation is built with Sphinx. -- 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]
