potiuk commented on code in PR #74825:
URL: https://github.com/apache/airflow/pull/74825#discussion_r4330956947
##########
contributing-docs/05_pull_requests.rst:
##########
@@ -94,8 +94,13 @@ these guidelines:
The Airflow repo uses `GitHub Actions
<https://help.github.com/en/actions>`_ to
run the tests and `codecov <https://codecov.io/gh/apache/airflow>`_ to
track
- coverage. You can set up both for free on your fork. It will help you make
sure you do not
- break the build with your PR and that you help increase coverage.
+ coverage. Unless you are a committer, a collaborator or a member of an
exempted organisation,
Review Comment:
Fixed. `05_pull_requests.rst` and "CI in your fork" now say that fork CI
applies to pilot participants and `use fork ci` PRs until it's switched on for
everyone, and that enabling the workflow outside that runs CI twice. `breeze ci
audit` tells you which case you're in. Also from a second review: the
maintainer docs now say that adding `use project ci` doesn't start project CI
by itself. Re-run the PR's latest `Tests (AMD)` run, or have the author push.
---
Drafted-by: Claude Code (Opus 5); reviewed by @potiuk before posting
--
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]
