The low hanging fruit.

Author: bvarner

README.md

@@ -203,7 +203,7 @@ plugins in the build process (deploy, site-deploy, etc.) will use the repositori
 | releaseDeploymentRepository | n/a | The repository to use for releases. (Builds with a GIT_BRANCH matching `masterBranchPattern` or `supportBranchPattern`) |
 | stageDeploymentRepository | n/a | The repository to use for staging. (Builds with a GIT_BRANCH matching `releaseBranchPattern` or `hotfixBranchPattern`) | 
 | snapshotDeploymentRepository | n/a | The repository to use for snapshots. (Builds matching `developmentBranchPattern`) |
-| otherDeploymentBranchPattern | false | N/A | Regex. When matched, the branch name is normalized and any artifacts produced by the build will include the normalized branch name and -SNAPSHOT. Deployment will target the snapshot repository|
+| otherDeploymentBranchPattern | n/a | Regex. When matched, the branch name is normalized and any artifacts produced by the build will include the normalized branch name and -SNAPSHOT. Deployment will target the snapshot repository |
 
 **The repository properties should follow the following format**, `id::layout::url::uniqueVersion`.
 

src/main/java/com/e_gineering/maven/gitflowhelper/RetargetDeployMojo.java

@@ -35,15 +35,12 @@ protected void execute(final GitBranchInfo gitBranchInfo) throws MojoExecutionEx
                 break;
             }
             case OTHER: {
-                // If other branches are set to deploy
-                // Other branches never target release, but may target stage for non-SNAPSHOT artifacts.
-                // For this reason, "overwrite" is considered _highly_ dangerous.
                 String otherBranchesToDeploy = resolveExpression(otherDeployBranchPattern);
 	            if (!"".equals(otherBranchesToDeploy) && gitBranchInfo.getName().matches(otherBranchesToDeploy)) {
                     setTargetSnapshots();
 
                     String branchName = gitBranchInfo.getName();
-                    String semVerAddition = "+" + branchName.replaceAll("[^0-9^A-Z^a-z^-^.]", "-") + "-SNAPSHOT";
+                    String semVerAddition = "+" + branchName.replaceAll("[^0-9A-Za-z-.]", "-") + "-SNAPSHOT";
 
                     updateArtifactVersion(project.getArtifact(), semVerAddition);
                     for (Artifact a : project.getAttachedArtifacts()) {

src/test/resources/project-alt1-stub/pom.xml

@@ -56,7 +56,7 @@
 					<releaseBranchProperties>
 						<foo>bar</foo>
 					</releaseBranchProperties>
-					<otherDeployBranchPattern>(origin/)feature/poc/.*</otherDeployBranchPattern>
+					<otherDeployBranchPattern>(origin/)?feature/poc/.*</otherDeployBranchPattern>
 				</configuration>
 				<executions>
 					<execution>