Skip to content

Disable recog-parent and build-tools deploy#23

Merged
mkienow-r7 merged 1 commit intorapid7:masterfrom
mkienow-r7:feature/skip-deploy
May 20, 2022
Merged

Disable recog-parent and build-tools deploy#23
mkienow-r7 merged 1 commit intorapid7:masterfrom
mkienow-r7:feature/skip-deploy

Conversation

@mkienow-r7
Copy link
Copy Markdown
Contributor

@mkienow-r7 mkienow-r7 commented May 20, 2022

Description

Disables deploy for recog-parent and build-tools modules.

Motivation and Context

To allow jenkins to produce a successful build.

How Has This Been Tested?

  • Modified distributionManagement block in root pom.xml
diff --git a/pom.xml b/pom.xml
index fe74d0b..9753067 100644
--- a/pom.xml
+++ b/pom.xml
@@ -31,14 +31,14 @@
   </licenses>

   <distributionManagement>
-    <snapshotRepository>
-      <id>ossrh</id>
-      <url>https://oss.sonatype.org/content/repositories/snapshots</url>
-    </snapshotRepository>
     <repository>
-      <id>ossrh</id>
-      <url>https://oss.sonatype.org/service/local/staging/deploy/maven2/</url>
+      <id>local-release</id>
+      <url>file:../local_repo/release</url>
     </repository>
+    <snapshotRepository>
+      <id>local-snapshot</id>
+      <url>file:../local_repo/snapshot</url>
+    </snapshotRepository>
   </distributionManagement>
  • Ran most of the goal executed in the job: mvn deploy -B -C -U -Dprod
  • Confirmed command ran without errors and that the recog-java and recog-verify modules were deployed to the local directory specified in the modified distributionManagement block

Types of changes

  • Bug fix (non-breaking change which fixes an issue)

Checklist:

  • I have updated the documentation accordingly (or changes are not required).
  • I have added tests to cover my changes (or new tests are not required).
  • All new and existing tests passed.

Copy link
Copy Markdown

@infosecci-r7 infosecci-r7 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Only pom.xml has changed, approved!

@mkienow-r7 mkienow-r7 merged commit 3709405 into rapid7:master May 20, 2022
@mkienow-r7 mkienow-r7 deleted the feature/skip-deploy branch May 20, 2022 20:06
@mkienow-r7 mkienow-r7 mentioned this pull request May 25, 2022
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants