wrap the enforcer rule with a profile that is activated by default

Author: suztomo

google-cloud-bigtable/pom.xml

@@ -349,6 +349,69 @@
   </dependencies>
 
   <profiles>
+    <profile>
+      <id>enable-enforcer-rules</id>
+      <activation>
+        <property>
+          <!-- By default, Maven runs the enforcer rules. The shared release script
+            may pass profile name that doesn't exist in this project. Let's ignore
+            the enforcer rule (requireProfileIdsExist) in that case, leveraging
+            the presence of the performRelease property. -->
+          <name>!performRelease</name>
+        </property>
+      </activation>
+      <build>
+        <plugins>
+          <plugin>
+            <groupId>org.apache.maven.plugins</groupId>
+            <artifactId>maven-enforcer-plugin</artifactId>
+            <version>3.5.0</version>
+            <executions>
+              <!-- Prevent users from mistyping a profile name -->
+              <execution>
+                <id>enforce-valid-profile</id>
+                <configuration>
+                  <rules>
+                    <requireProfileIdsExist/>
+                  </rules>
+                </configuration>
+                <goals>
+                  <goal>enforce</goal>
+                </goals>
+              </execution>
+              <!-- Ensure that greatest version of each transitive dependency got selected -->
+              <execution>
+                <id>enforce-dependency-upper-bound</id>
+                <configuration>
+                  <rules>
+                    <requireUpperBoundDeps/>
+                  </rules>
+                </configuration>
+                <goals>
+                  <goal>enforce</goal>
+                </goals>
+              </execution>
+            </executions>
+            <!--
+              workaround for https://github.com/mojohaus/extra-enforcer-rules/issues/132
+              Remove when it's fixed upstream.
+            -->
+            <dependencies>
+              <dependency>
+                <groupId>org.codehaus.mojo</groupId>
+                <artifactId>extra-enforcer-rules</artifactId>
+                <version>1.8.0</version>
+              </dependency>
+              <dependency>
+                <groupId>org.apache.maven.shared</groupId>
+                <artifactId>maven-dependency-tree</artifactId>
+                <version>3.3.0</version>
+              </dependency>
+            </dependencies>
+          </plugin>
+        </plugins>
+      </build>
+    </profile>
     <profile>
       <id>enable-verbose-grpc-logs</id>
       <properties>
@@ -668,42 +731,6 @@
           </execution>
         </executions>
       </plugin>
-      <plugin>
-        <groupId>org.apache.maven.plugins</groupId>
-        <artifactId>maven-enforcer-plugin</artifactId>
-        <version>3.5.0</version>
-        <executions>
-          <!-- Ensure that greatest version of each transitive dependency got selected -->
-          <execution>
-            <id>enforce-dependency-upper-bound</id>
-            <configuration>
-              <rules>
-                <requireUpperBoundDeps/>
-              </rules>
-            </configuration>
-            <goals>
-              <goal>enforce</goal>
-            </goals>
-          </execution>
-        </executions>
-        <!--
-          workaround for https://github.com/mojohaus/extra-enforcer-rules/issues/132
-          Remove when it's fixed upstream.
-        -->
-        <dependencies>
-          <dependency>
-            <groupId>org.codehaus.mojo</groupId>
-            <artifactId>extra-enforcer-rules</artifactId>
-            <version>1.8.0</version>
-          </dependency>
-          <dependency>
-            <groupId>org.apache.maven.shared</groupId>
-            <artifactId>maven-dependency-tree</artifactId>
-            <version>3.3.0</version>
-          </dependency>
-        </dependencies>
-      </plugin>
-
       <plugin>
         <groupId>org.apache.maven.plugins</groupId>
         <artifactId>maven-dependency-plugin</artifactId>