Hello everyone, As discussed in https://lists.apache.org/thread/cs6mcvpn2lk9w2p4oz43t20z3fg5nl7l I just enabled "require conversation resolution" for our main/stable branches. We have not used it in the past so it might not work as we think or we might need to tweak something.
Generally speaking (if all works) all conversations on PRs should be resolved before we can merge the PR. This "resolving" is encouraged to be done by the author when they think the conversation is resolved, but it can also be done by reviewers or the maintainer who wants to merge the PR. We attempted to describe some basic rules and expectations here: https://github.com/apache/airflow/blob/main/CONTRIBUTING.rst#step-5-pass-pr-review but undoubtedly there will be questions and issues that we might want to solve - so feel free to discuss it here or raise question/issues in #development channel in slack (I am also happy to be pinged directly about it and help to resolve any issues/gather feedback). J.