potiuk commented on PR #65791: URL: https://github.com/apache/airflow/pull/65791#issuecomment-4473467291
@sokratis1988 A few things need addressing before review — see our [Pull Request quality criteria](https://github.com/apache/airflow/blob/main/contributing-docs/05_pull_requests.rst#pull-request-quality-criteria). **Issues found:** - :x: **Build documentation (spellcheck)**: `CI image checks / Build documentation (--spellcheck-only)` is failing. Run `breeze build-docs --spellcheck-only` locally to see which words it didn't accept, then either fix the spelling or add the term to `docs/spelling_wordlist.txt` if it's a legitimate technical word. See the [docs-building docs](https://github.com/apache/airflow/blob/main/contributing-docs/11_documentation_building.rst). **What to do next:** - Push a fix for the spellcheck failure. - @jscheffl asked a design question above ("Why would you want to override the ENV if there are specific helper template entries…") — please reply with the use case so the design conversation can move forward. There is no rush — take your time and work at your own pace. We appreciate your contribution and are happy to wait for updates. If you have questions, feel free to ask on the [Airflow Slack](https://s.apache.org/airflow-slack). --- _Note: This comment was drafted by an AI-assisted triage tool and may contain mistakes. Once you have addressed the points above, an Apache Airflow maintainer — a real person — will take the next look at your PR. We use this [two-stage triage process](https://github.com/apache/airflow/blob/main/contributing-docs/25_maintainer_pr_triage.md#why-the-first-pass-is-automated) so that our maintainers' limited time is spent where it matters most: the conversation with you._ -- 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]
