ashb commented on code in PR #74549:
URL: https://github.com/apache/airflow/pull/74549#discussion_r4248087280
##########
.github/workflows/publish-docs-to-s3.yml:
##########
@@ -375,7 +375,12 @@ jobs:
> generated/_build/docs/sphinx-build-skipped.txt
echo "sphinx-build-skipped=true" >> "${GITHUB_OUTPUT}"
else
- breeze build-docs ${SPHINX_INCLUDE_DOCS} --docs-only
${FAIL_ON_INVENTORIES}
+ # Refs whose breeze still has --docs-only check spelling in a
second full build; skip it.
+ DOCS_ONLY=""
+ if grep -q '"--docs-only"'
dev/breeze/src/airflow_breeze/commands/developer_commands.py; then
+ DOCS_ONLY="--docs-only"
+ fi
+ breeze build-docs ${SPHINX_INCLUDE_DOCS} ${DOCS_ONLY}
${FAIL_ON_INVENTORIES}
Review Comment:
Honestly it feels better to check spelling now -- we don't want spelling
mistakes in the published docs, and on the _odd_ chance that someone merged the
word and the wordlist separately I think we should have to cherry pick both.
--
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]
