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.

Reply via email to