From ffb230590403f1fa1c900000d4fa77c1daf9f169 Mon Sep 17 00:00:00 2001
From: Mitch Denny <midenn@microsoft.com>
Date: Fri, 17 Jul 2020 10:18:58 +1000
Subject: [PATCH 1/6] Try out build helper.

---
 sdk/parents/azure-client-sdk-parent/pom.xml | 24 ++++++++++++++++++++-
 1 file changed, 23 insertions(+), 1 deletion(-)

diff --git a/sdk/parents/azure-client-sdk-parent/pom.xml b/sdk/parents/azure-client-sdk-parent/pom.xml
index 56026116ef6a..9eed63baf762 100644
--- a/sdk/parents/azure-client-sdk-parent/pom.xml
+++ b/sdk/parents/azure-client-sdk-parent/pom.xml
@@ -388,6 +388,29 @@
         <groupId>org.codehaus.mojo</groupId>
         <artifactId>build-helper-maven-plugin</artifactId>
         <version>3.0.0</version> <!-- {x-version-update;org.codehaus.mojo:build-helper-maven-plugin;external_dependency} -->
+        <executions>
+            <execution>
+            <id>attach-artifacts</id>
+            <phase>package</phase>
+            <goals>
+                <goal>attach-artifact</goal>
+            </goals>
+            <configuration>
+                <artifacts>
+                  <artifact>
+                      <file>${basedir}/README.md</file>
+                      <type>md</type>
+                      <classifier>readme</classifier>
+                  </artifact>
+                  <artifact>
+                    <file>${basedir}/CHANGELOG.md</file>
+                    <type>md</type>
+                    <classifier>changelog</classifier>
+                </artifact>
+              </artifacts>
+            </configuration>
+            </execution>
+        </executions>
       </plugin>
 
       <!-- Checks public surface area for breaking changes. -->
@@ -687,7 +710,6 @@
             <quiet>true</quiet>
           </configuration>
         </plugin>
-
         <plugin>
           <groupId>com.github.spotbugs</groupId>
           <artifactId>spotbugs-maven-plugin</artifactId>

From b8c24ebb88017737c4775a56cc843b9c5fe8c84d Mon Sep 17 00:00:00 2001
From: Mitch Denny <midenn@microsoft.com>
Date: Fri, 17 Jul 2020 10:45:04 +1000
Subject: [PATCH 2/6] Remove copy steps, don't think they are necessary now.

---
 sdk/parents/azure-client-sdk-parent/pom.xml | 70 ---------------------
 1 file changed, 70 deletions(-)

diff --git a/sdk/parents/azure-client-sdk-parent/pom.xml b/sdk/parents/azure-client-sdk-parent/pom.xml
index 9eed63baf762..20135c07ff11 100644
--- a/sdk/parents/azure-client-sdk-parent/pom.xml
+++ b/sdk/parents/azure-client-sdk-parent/pom.xml
@@ -1265,76 +1265,6 @@
       </build>
     </profile>
 
-    <profile>
-      <id>changelog</id>
-      <activation>
-        <file>
-          <exists>${basedir}/CHANGELOG.md</exists>
-        </file>
-      </activation>
-      <build>
-        <plugins>
-          <plugin>
-            <groupId>org.apache.maven.plugins</groupId>
-            <artifactId>maven-antrun-plugin</artifactId>
-            <version>1.8</version> <!-- {x-version-update;org.apache.maven.plugins:maven-antrun-plugin;external_dependency} -->
-            <executions>
-              <execution>
-                <id>copy-changelog</id>
-                <phase>package</phase>
-                <configuration>
-                  <target>
-                    <echo>Copying ${project.basedir}/CHANGELOG.md to
-                      ${packageOutputDirectory}/${project.build.finalName}-changelog.md
-                    </echo>
-                    <copy file="${project.basedir}/CHANGELOG.md" tofile="${packageOutputDirectory}/${project.build.finalName}-changelog.md"/>
-                  </target>
-                </configuration>
-                <goals>
-                  <goal>run</goal>
-                </goals>
-              </execution>
-            </executions>
-          </plugin>
-        </plugins>
-      </build>
-    </profile>
-
-    <profile>
-      <id>readme</id>
-      <activation>
-        <file>
-          <exists>${basedir}/README.md</exists>
-        </file>
-      </activation>
-      <build>
-        <plugins>
-          <plugin>
-            <groupId>org.apache.maven.plugins</groupId>
-            <artifactId>maven-antrun-plugin</artifactId>
-            <version>1.8</version> <!-- {x-version-update;org.apache.maven.plugins:maven-antrun-plugin;external_dependency} -->
-            <executions>
-              <execution>
-                <id>copy-readme</id>
-                <phase>package</phase>
-                <configuration>
-                  <target>
-                    <echo>Copying ${project.basedir}/README.md to
-                      ${packageOutputDirectory}/${project.build.finalName}-readme.md
-                    </echo>
-                    <copy file="${project.basedir}/README.md" tofile="${packageOutputDirectory}/${project.build.finalName}-readme.md"/>
-                  </target>
-                </configuration>
-                <goals>
-                  <goal>run</goal>
-                </goals>
-              </execution>
-            </executions>
-          </plugin>
-        </plugins>
-      </build>
-    </profile>
-
     <profile>
       <!-- separate profile for windows as the executable on windows is named npx.cmd and npx on other os families -->
       <id>readme-codesnippet-windows</id>

From 087a86966ad08cf0ba01100734909dcb7523faca Mon Sep 17 00:00:00 2001
From: Mitch Denny <midenn@microsoft.com>
Date: Fri, 17 Jul 2020 12:43:44 +1000
Subject: [PATCH 3/6] Add changelog for perf-test-core.

---
 common/perf-test-core/CHANGELOG.md | 3 +++
 1 file changed, 3 insertions(+)
 create mode 100644 common/perf-test-core/CHANGELOG.md

diff --git a/common/perf-test-core/CHANGELOG.md b/common/perf-test-core/CHANGELOG.md
new file mode 100644
index 000000000000..6ced156fc49d
--- /dev/null
+++ b/common/perf-test-core/CHANGELOG.md
@@ -0,0 +1,3 @@
+# Release History
+
+## 1.0.0-beta.1 (Unreleased)
\ No newline at end of file

From baf4b67b6a1ed87dfedfbaf69e4bfb846d373780 Mon Sep 17 00:00:00 2001
From: Mitch Denny <midenn@microsoft.com>
Date: Fri, 17 Jul 2020 12:48:12 +1000
Subject: [PATCH 4/6] Allow skip on missing files.

---
 common/perf-test-core/CHANGELOG.md          | 3 ---
 sdk/parents/azure-client-sdk-parent/pom.xml | 1 +
 2 files changed, 1 insertion(+), 3 deletions(-)
 delete mode 100644 common/perf-test-core/CHANGELOG.md

diff --git a/common/perf-test-core/CHANGELOG.md b/common/perf-test-core/CHANGELOG.md
deleted file mode 100644
index 6ced156fc49d..000000000000
--- a/common/perf-test-core/CHANGELOG.md
+++ /dev/null
@@ -1,3 +0,0 @@
-# Release History
-
-## 1.0.0-beta.1 (Unreleased)
\ No newline at end of file
diff --git a/sdk/parents/azure-client-sdk-parent/pom.xml b/sdk/parents/azure-client-sdk-parent/pom.xml
index 20135c07ff11..ba538c26fb58 100644
--- a/sdk/parents/azure-client-sdk-parent/pom.xml
+++ b/sdk/parents/azure-client-sdk-parent/pom.xml
@@ -408,6 +408,7 @@
                     <classifier>changelog</classifier>
                 </artifact>
               </artifacts>
+              <skipAttach>true</skipAttach> <!-- If the files don't exist, don't blow up. -->
             </configuration>
             </execution>
         </executions>

From 59146ddb526b944f6181df86a73eb65e03671ffc Mon Sep 17 00:00:00 2001
From: Mitch Denny <midenn@microsoft.com>
Date: Fri, 17 Jul 2020 13:33:04 +1000
Subject: [PATCH 5/6] Putting missing changelogs and readmes in.

---
 common/perf-test-core/CHANGELOG.md            | 3 +++
 eng/jacoco-test-coverage/CHANGELOG.md         | 3 +++
 eng/jacoco-test-coverage/README.md            | 2 ++
 sdk/e2e/CHANGELOG.md                          | 3 +++
 sdk/parents/azure-client-sdk-parent/README.md | 1 +
 5 files changed, 12 insertions(+)
 create mode 100644 common/perf-test-core/CHANGELOG.md
 create mode 100644 eng/jacoco-test-coverage/CHANGELOG.md
 create mode 100644 eng/jacoco-test-coverage/README.md
 create mode 100644 sdk/e2e/CHANGELOG.md
 create mode 100644 sdk/parents/azure-client-sdk-parent/README.md

diff --git a/common/perf-test-core/CHANGELOG.md b/common/perf-test-core/CHANGELOG.md
new file mode 100644
index 000000000000..4144f75694a0
--- /dev/null
+++ b/common/perf-test-core/CHANGELOG.md
@@ -0,0 +1,3 @@
+# Release History
+
+## 1.0.0-beta.1 (Unreleased)
diff --git a/eng/jacoco-test-coverage/CHANGELOG.md b/eng/jacoco-test-coverage/CHANGELOG.md
new file mode 100644
index 000000000000..04a5fd6d8dc5
--- /dev/null
+++ b/eng/jacoco-test-coverage/CHANGELOG.md
@@ -0,0 +1,3 @@
+# Release History
+
+## 1.0.0-SNAPSHOT (Unreleased)
diff --git a/eng/jacoco-test-coverage/README.md b/eng/jacoco-test-coverage/README.md
new file mode 100644
index 000000000000..56ddda8e4e3d
--- /dev/null
+++ b/eng/jacoco-test-coverage/README.md
@@ -0,0 +1,2 @@
+# Microsoft Azure Client Library - Test coverage
+Nothing exciting to see here, this just aggregates the modules for reporting purposes.
\ No newline at end of file
diff --git a/sdk/e2e/CHANGELOG.md b/sdk/e2e/CHANGELOG.md
new file mode 100644
index 000000000000..4144f75694a0
--- /dev/null
+++ b/sdk/e2e/CHANGELOG.md
@@ -0,0 +1,3 @@
+# Release History
+
+## 1.0.0-beta.1 (Unreleased)
diff --git a/sdk/parents/azure-client-sdk-parent/README.md b/sdk/parents/azure-client-sdk-parent/README.md
new file mode 100644
index 000000000000..b7adbdce9468
--- /dev/null
+++ b/sdk/parents/azure-client-sdk-parent/README.md
@@ -0,0 +1 @@
+# Microsoft Azure SDK for Java - Client Libraries
\ No newline at end of file

From 2244a1a33f74c65cf53213f7756e70d7b3b5aa11 Mon Sep 17 00:00:00 2001
From: Mitch Denny <midenn@microsoft.com>
Date: Fri, 17 Jul 2020 13:48:16 +1000
Subject: [PATCH 6/6] Remove config option, add content issue exclusion.

---
 eng/.docsettings.yml                        | 1 +
 sdk/parents/azure-client-sdk-parent/pom.xml | 1 -
 2 files changed, 1 insertion(+), 1 deletion(-)

diff --git a/eng/.docsettings.yml b/eng/.docsettings.yml
index e423fcddfc81..b99742d8a0bc 100644
--- a/eng/.docsettings.yml
+++ b/eng/.docsettings.yml
@@ -131,6 +131,7 @@ known_content_issues:
   - ['sdk/formrecognizer/azure-ai-formrecognizer/swagger/README.md', '#3113']
   - ['sdk/keyvault/README.md', '#3113']
   - ['sdk/loganalytics/microsoft-azure-loganalytics/README.md', '#3113']
+  - ['sdk/parents/azure-client-sdk-parent/README.md', '#3113']
   - ['sdk/search/azure-search-documents/swagger/readme.md', '#3113']
   - ['sdk/search/azure-search-documents/src/test/README.md', '#3113']
   - ['sdk/search/open-api/readme.md', '#3113']
diff --git a/sdk/parents/azure-client-sdk-parent/pom.xml b/sdk/parents/azure-client-sdk-parent/pom.xml
index ba538c26fb58..20135c07ff11 100644
--- a/sdk/parents/azure-client-sdk-parent/pom.xml
+++ b/sdk/parents/azure-client-sdk-parent/pom.xml
@@ -408,7 +408,6 @@
                     <classifier>changelog</classifier>
                 </artifact>
               </artifacts>
-              <skipAttach>true</skipAttach> <!-- If the files don't exist, don't blow up. -->
             </configuration>
             </execution>
         </executions>