GitHub provides some tools to answer this kind of question.

Step 1: Under the PR's "Files Changed" tab, click File Filter > Your CODEOWNER 
files
Step 2: Hover over the keyhole/shield icon (to the left of the red/green 
changecount graphic to the left of each filename) to see who else is also owner.
Step 3: If the ownership doesn't seem quite right, click on the keyhole/shield 
icon to see the exact line in CODEOWNERS that was applied.

Please note, if a file has NO codeowner, it will still show up in the "Your 
CODEOWNER files" (but no keyhole/shield indicator will be displayed).  It would 
be fantastic to find owners for these remaining unowned files, but until then 
owned-by-no-one == owned-by-everyone.

Improvements to CODEOWNERS can always be submitted via PR.

Some files in your list below have no owner, but several are owned by someone 
else so they shouldn't be showing up in your list.  Perhaps you copy&pasted 
this list from the Jump To dropdown?  If so, it seems that a File Filter 
selection does not filter the Jump To dropdown, only the display below.

On 2/19/21, 11:30 AM, "Bruce Schuchardt" <bru...@vmware.com> wrote:

    I was pulled in to PR 5989 but can’t figure out how that happened with the 
current CODEOWNERS file.  These all seem out of my area:

    boms/geode-all-bom/src/test/resources/expected-pom.xml
      
buildSrc/src/main/groovy/org/apache/geode/gradle/plugins/DependencyConstraints.groovy
     extensions/geode-modules-session/build.gradle
     extensions/geode-modules/build.gradle
      extensions/geode-modules/src/test/resources/expected-pom.xml
     extensions/session-testing-war/build.gradle
      
...embly/geode-assembly-test/src/main/java/org/apache/geode/session/tests/TomcatInstall.java
       
geode-assembly/src/acceptanceTest/java/org/apache/geode/modules/DeployJarAcceptanceTest.java
     
...butedTest/java/org/apache/geode/management/internal/rest/DeployToMultiGroupDUnitTest.java
      
...tedTest/java/org/apache/geode/management/internal/rest/DeploymentManagementDUnitTest.java
     
.../java/org/apache/geode/management/internal/rest/DeploymentManagementRedployDUnitTest.java
      
...java/org/apache/geode/management/internal/rest/DeploymentSemanticVersionJarDUnitTest.java
      geode-assembly/src/integrationTest/resources/assembly_content.txt
     geode-assembly/src/integrationTest/resources/dependency_classpath.txt
     
.../java/org/apache/geode/connectors/jdbc/internal/cli/CreateDataSourceCommandDUnitTest.java
     geode-core/build.gradle
      
...a/org/apache/geode/management/internal/configuration/ClusterConfigDeployJarDUnitTest.java
      
geode-core/src/integrationTest/java/org/apache/geode/internal/ClassPathLoaderDeployTest.java

Reply via email to