From ef370637da1cdf40b0a6309f9f5049f8f9681f91 Mon Sep 17 00:00:00 2001
From: Spencer <email@spalger.com>
Date: Wed, 7 Oct 2020 16:52:45 -0700
Subject: [PATCH] [release notes] extract "dev docs" comment too (#79351)
 (#79945)

Co-authored-by: spalger <spalger@users.noreply.github.com>
Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>

Co-authored-by: spalger <spalger@users.noreply.github.com>
Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>
---
 docs/developer/contributing/index.asciidoc       |  2 +-
 .../src/lib/get_note_from_description.test.ts    | 16 ++++++++++------
 .../src/lib/get_note_from_description.ts         |  5 +++--
 packages/kbn-release-notes/src/lib/pr_api.ts     |  4 +++-
 4 files changed, 17 insertions(+), 10 deletions(-)

diff --git a/docs/developer/contributing/index.asciidoc b/docs/developer/contributing/index.asciidoc
index ecb37ffe9c97..ba4ab89d17c2 100644
--- a/docs/developer/contributing/index.asciidoc
+++ b/docs/developer/contributing/index.asciidoc
@@ -49,7 +49,7 @@ The Release Notes summarize what the PRs accomplish in language that is meaningf
 
 The text that appears in the Release Notes is pulled directly from your PR title, or a single paragraph of text that you specify in the PR description.
 
-To use a single paragraph of text, enter `Release note:` or a `## Release note` header in the PR description, followed by your text. For example, refer to this https://github.com/elastic/kibana/pull/65796[PR] that uses the `## Release note` header.
+To use a single paragraph of text, enter a `Release note:` or `## Release note` header in the PR description ("dev docs" works too), followed by your text. For example, refer to this https://github.com/elastic/kibana/pull/65796[PR] that uses the `## Release note` header.
 
 When you create the Release Notes text, use the following best practices:
 
diff --git a/packages/kbn-release-notes/src/lib/get_note_from_description.test.ts b/packages/kbn-release-notes/src/lib/get_note_from_description.test.ts
index 23dcb302f090..22b9713b7833 100644
--- a/packages/kbn-release-notes/src/lib/get_note_from_description.test.ts
+++ b/packages/kbn-release-notes/src/lib/get_note_from_description.test.ts
@@ -35,7 +35,8 @@ it('extracts expected components from html', () => {
         ## Release Note:
 
         Checkout this feature
-      `)
+      `),
+      'release note'
     )
   ).toMatchInlineSnapshot(`"Checkout this feature"`);
 
@@ -46,10 +47,11 @@ it('extracts expected components from html', () => {
 
         Fixes: #1234
 
-        #### Release Note:
+        #### Dev docs:
 
         We fixed an issue
-      `)
+      `),
+      'dev docs'
     )
   ).toMatchInlineSnapshot(`"We fixed an issue"`);
 
@@ -60,8 +62,9 @@ it('extracts expected components from html', () => {
 
         Fixes: #1234
 
-        Release note: Checkout feature foo
-      `)
+        OTHER TITLE: Checkout feature foo
+      `),
+      'other title'
     )
   ).toMatchInlineSnapshot(`"Checkout feature foo"`);
 
@@ -73,7 +76,8 @@ it('extracts expected components from html', () => {
         My PR description
 
            release note : bar
-      `)
+      `),
+      'release note'
     )
   ).toMatchInlineSnapshot(`"bar"`);
 });
diff --git a/packages/kbn-release-notes/src/lib/get_note_from_description.ts b/packages/kbn-release-notes/src/lib/get_note_from_description.ts
index 57df203470a5..0d9135c431e3 100644
--- a/packages/kbn-release-notes/src/lib/get_note_from_description.ts
+++ b/packages/kbn-release-notes/src/lib/get_note_from_description.ts
@@ -19,11 +19,12 @@
 
 import cheerio from 'cheerio';
 
-export function getNoteFromDescription(descriptionHtml: string) {
+export function getNoteFromDescription(descriptionHtml: string, header: string) {
+  const re = new RegExp(`^(\\s*${header.toLowerCase()}(?:s)?\\s*:?\\s*)`, 'i');
   const $ = cheerio.load(descriptionHtml);
   for (const el of $('p,h1,h2,h3,h4,h5').toArray()) {
     const text = $(el).text();
-    const match = text.match(/^(\s*release note(?:s)?\s*:?\s*)/i);
+    const match = text.match(re);
 
     if (!match) {
       continue;
diff --git a/packages/kbn-release-notes/src/lib/pr_api.ts b/packages/kbn-release-notes/src/lib/pr_api.ts
index 1f26aa7ad86c..5fa3dfdba10e 100644
--- a/packages/kbn-release-notes/src/lib/pr_api.ts
+++ b/packages/kbn-release-notes/src/lib/pr_api.ts
@@ -178,7 +178,9 @@ export class PrApi {
       versions: labels
         .map((l) => Version.fromLabel(l))
         .filter((v): v is Version => v instanceof Version),
-      note: getNoteFromDescription(node.bodyHTML),
+      note:
+        getNoteFromDescription(node.bodyHTML, 'release note') ||
+        getNoteFromDescription(node.bodyHTML, 'dev docs'),
     };
   }