This is a very good point :) and has surprising (at least to me) answer: We have two mechanisms now:
* Labels by Boring Cyborg (not very helpful as you cannot selectively subscribe to a label): https://github.com/apache/airflow/blob/main/.github/boring-cyborg.yml * CODEOWNERS feature (this one is precisely what you want, because you are marked as reviewer and notified as CODEOWNER): https://github.com/apache/airflow/blob/main/.github/CODEOWNERS The problem with CODEOWNERS is (or rather was(!) ) that you could only add maintainers as CODEOWNERS. Up until recently the CODEOWNER documentation stated this: > The people you choose as code owners must have write permissions for the repository. When the code owner is a team, that team must have write permissions, even if all the individual members of the team already have write permissions directly, through organization membership, or through another team membership. However it seems that GitHub silently changed it :) and now it's a bit different: > Users must have *read* access to the repository and teams must have explicit write access, even if the team’s members already have access. You can also refer to a user by an email address that has been added to their account on GitHub.com, for example u...@example.com. So - seems you can become a CODEOWNER even if you are not a maintainer!!!! YAY!. Something we really wanted to add for providers for a long time! I think you can achieve PRECISELY what you want - just make a PR to CODEOWNERS and add yourself to all Google Provider paths and we can test it. J. On Mon, Apr 11, 2022 at 10:07 AM Bartłomiej Hirsz <bartek.hi...@gmail.com> wrote: > Hi, > > Do we have any mechanism in Airflow to be notified on PRs touching > provider code hosted in the Airflow repository? I think that's not the case > - and it's related to recent discussions about providers in the core > repository (on dev list). I've had some problems in the past where I > discovered that there were PRs that changed behaviour of our provider and > it would be useful to be noticed on such PRs. > For example I've raised PR migrating Google GCS system tests to new design: > https://github.com/apache/airflow/pull/22778 (5 days ago) > > Then someone raised PR extending the same files, but using old design: > https://github.com/apache/airflow/pull/22808 (4 days ago) > > The latter is already merged - now causing headache if it comes to > properly merging the changes. It could be avoided if there would be a code > owner set per files. Is it possible in Airflow? I know there is an > ownership file but previously only commiters (people with write access) > could be added. > > Best regards, > Bartłomiej Hirsz >
