commit 2c05d28c50ccf4ba5be42c0baf6df1be4f5e3075
author Jungho Ahn <pobby21@gmail.com> Thu Jan 03 10:51:51 2019 -0800
committer Luca Milanesio <luca.milanesio@gmail.com> Wed May 08 20:29:37 2019 +0000
tree 99bae05261e8b20360253b87faee40e594a91c16
parent 6bbd5a5f611c455e831344619e556d15ae141a94

Change the default inheritance to true

The plugin document says "inherited" is true by default:

"The plugin analyzes the latest patch set by looking at each file
directory and building an OWNERS hierarchy. It stops once it finds an
OWNERS file that has “inherited” set to false (by default it’s true.)"

This change changes the default value to true to match the document.

Bug: Issue 10199
Change-Id: I929c46acfa89637ab13cb950e7c6ad5a34531944

owners-common/src/main/java/com/googlesource/gerrit/owners/common/ConfigurationParser.java

diff --git a/owners-common/src/main/java/com/googlesource/gerrit/owners/common/ConfigurationParser.java b/owners-common/src/main/java/com/googlesource/gerrit/owners/common/ConfigurationParser.java
index 2f5c5c0..dcdd9f7 100644
--- a/owners-common/src/main/java/com/googlesource/gerrit/owners/common/ConfigurationParser.java
+++ b/owners-common/src/main/java/com/googlesource/gerrit/owners/common/ConfigurationParser.java
@@ -43,7 +43,7 @@
       final OwnersConfig ret = new OwnersConfig();
       JsonNode jsonNode = new ObjectMapper(new YAMLFactory()).readValue(yamlBytes, JsonNode.class);
       Boolean inherited =
-          Optional.ofNullable(jsonNode.get("inherited")).map(JsonNode::asBoolean).orElse(false);
+          Optional.ofNullable(jsonNode.get("inherited")).map(JsonNode::asBoolean).orElse(true);
       ret.setInherited(inherited);
       addClassicMatcher(jsonNode, ret);
       addMatchers(jsonNode, ret);