You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by GitBox <gi...@apache.org> on 2021/02/11 16:15:40 UTC
[GitHub] [airflow] RNHTTR opened a new pull request #14196: Reorder doc/spelling build order and improve spelling error message for CI
RNHTTR opened a new pull request #14196:
URL: https://github.com/apache/airflow/pull/14196
Generic Sphinx errors that were unrelated to spelling were being raised as spelling errors leading to potential confusion. Reversing the order of the build (i.e. running the docs build before the spelling build) will catch such errors and more appropriately denote the issue as a build issue and not a spelling issue. Additionally, the error message for Sphinx errors unrelated to spelling has been updated to be more clear for such cases.
closes: #14051
----------------------------------------------------------------
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:
users@infra.apache.org
[GitHub] [airflow] kaxil merged pull request #14196: Reorder doc/spelling build order and improve spelling error message for CI
Posted by GitBox <gi...@apache.org>.
kaxil merged pull request #14196:
URL: https://github.com/apache/airflow/pull/14196
----------------------------------------------------------------
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:
users@infra.apache.org
[GitHub] [airflow] kaxil commented on pull request #14196: Reorder doc/spelling build order and improve spelling error message for CI
Posted by GitBox <gi...@apache.org>.
kaxil commented on pull request #14196:
URL: https://github.com/apache/airflow/pull/14196#issuecomment-777634548
Thanks @RNHTTR for the PR
----------------------------------------------------------------
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:
users@infra.apache.org
[GitHub] [airflow] RNHTTR commented on pull request #14196: Reorder doc/spelling build order and improve spelling error message for CI
Posted by GitBox <gi...@apache.org>.
RNHTTR commented on pull request #14196:
URL: https://github.com/apache/airflow/pull/14196#issuecomment-778482439
How ironic! 🤦 Sorry about that!
----------------------------------------------------------------
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:
users@infra.apache.org
[GitHub] [airflow] RNHTTR commented on pull request #14196: Reorder doc/spelling build order and improve spelling error message for CI
Posted by GitBox <gi...@apache.org>.
RNHTTR commented on pull request #14196:
URL: https://github.com/apache/airflow/pull/14196#issuecomment-778336113
@ashb @potiuk
----------------------------------------------------------------
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:
users@infra.apache.org
[GitHub] [airflow] RNHTTR commented on pull request #14196: Reorder doc/spelling build order and improve spelling error message for CI
Posted by GitBox <gi...@apache.org>.
RNHTTR commented on pull request #14196:
URL: https://github.com/apache/airflow/pull/14196#issuecomment-778510605
Fixed :)
----------------------------------------------------------------
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:
users@infra.apache.org
[GitHub] [airflow] potiuk commented on pull request #14196: Reorder doc/spelling build order and improve spelling error message for CI
Posted by GitBox <gi...@apache.org>.
potiuk commented on pull request #14196:
URL: https://github.com/apache/airflow/pull/14196#issuecomment-778351870
There is a (ups!) spelling error in the build :)
----------------------------------------------------------------
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:
users@infra.apache.org
[GitHub] [airflow] github-actions[bot] commented on pull request #14196: Reorder doc/spelling build order and improve spelling error message for CI
Posted by GitBox <gi...@apache.org>.
github-actions[bot] commented on pull request #14196:
URL: https://github.com/apache/airflow/pull/14196#issuecomment-777634847
The PR is likely ready to be merged. No tests are needed as no important environment files, nor python files were modified by it. However, committers might decide that full test matrix is needed and add the 'full tests needed' label. Then you should rebase it to the latest master or amend the last commit of the PR, and push it with --force-with-lease.
----------------------------------------------------------------
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:
users@infra.apache.org