[Jenkins] run CI when plugin readmes change (#74388)

Spencer · spalger · spalger · commit 9ae720b5a14e · 2020-08-05T12:06:15.000-07:00
Co-authored-by: spalger &lt;spalger@users.noreply.github.com&gt;
diff --git a/.ci/pipeline-library/src/test/prChanges.groovy b/.ci/pipeline-library/src/test/prChanges.groovy
@@ -97,4 +97,17 @@ class PrChangesTest extends KibanaBasePipelineTest {
 
     assertFalse(prChanges.areChangesSkippable())
   }
+
+  @Test
+  void 'areChangesSkippable() with plugin readme changes'() {
+    props([
+      githubPrs: [
+        getChanges: { [
+          [filename: 'src/plugins/foo/README.asciidoc'],
+        ] },
+      ],
+    ])
+
+    assertFalse(prChanges.areChangesSkippable())
+  }
 }
diff --git a/vars/prChanges.groovy b/vars/prChanges.groovy
@@ -22,6 +22,8 @@ def getNotSkippablePaths() {
   return [
     // this file is auto-generated and changes to it need to be validated with CI
     /^docs\/developer\/architecture\/code-exploration.asciidoc$/,
+    // don't skip CI on prs with changes to plugin readme files (?i) is for case-insensitive matching
+    /(?i)\/plugins\/[^\/]+\/readme\.(md|asciidoc)$/,
   ]
 }
 

Original file line number	Diff line number	Diff line change
`@@ -22,6 +22,8 @@ def getNotSkippablePaths() {`
`22`	`22`	`return [`
`23`	`23`	`// this file is auto-generated and changes to it need to be validated with CI`
`24`	`24`	`/^docs\/developer\/architecture\/code-exploration.asciidoc$/,`
	`25`	`+ // don't skip CI on prs with changes to plugin readme files (?i) is for case-insensitive matching`
	`26`	`+ /(?i)\/plugins\/[^\/]+\/readme\.(md\|asciidoc)$/,`
`25`	`27`	`]`
`26`	`28`	`}`
`27`	`29`