The GitHub Actions job "Tests" on airflow.git has succeeded. Run started by GitHub user potiuk (triggered by potiuk).
Head commit for run: f28b71131949191d9bb07e88f167517375049491 / Jarek Potiuk <[email protected]> Review and update our Pull Request review process and policy While applying result of the discussion about "resolving conversation" I have found that it would be good - while enabling it to test it to describe a bit better and explain some of the things we learned and "practice" for our PRs, so that we can easily refer to it when contributors ask us what to expect when they opened a PR. This change introduces the following: * explains the contributors what they can expect during PR review * explains what expectations we have for such PRs and what is the role of author in fulfilling the expectations and following up * explain how to treat conversations and how using conversations and expecting them to be resolved is a sign of trust * explains that we experimentally enable the "resolve conversation" requirement and why we are doing it, and what we expect to get from it * adds a few PR guidelines that we often discuss and explain to the contributors but we did not have them written down. * cleans up protected_branches configuration in the .asf.yaml and enables "resolve conversation" requirement for `main` branch * it also adds "linear history" requirement for all our `main` and `stable` brances - that was implicitly protected also by our merge policies, but enabling it explicitly is better protection for case where someone would enable "merge" approach temporarily for other branches (maybe). Report URL: https://github.com/apache/airflow/actions/runs/7350476719 With regards, GitHub Actions via GitBox --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
