Skip to content

Correct SCM tag after 0.8.1 build issues#27

Merged
gschneider-r7 merged 1 commit intorapid7:masterfrom
mkienow-r7:bug/correct-pom-scm-tag
May 26, 2022
Merged

Correct SCM tag after 0.8.1 build issues#27
gschneider-r7 merged 1 commit intorapid7:masterfrom
mkienow-r7:bug/correct-pom-scm-tag

Conversation

@mkienow-r7
Copy link
Copy Markdown
Contributor

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

Description

The release build for 0.8.1 encountered issues and between Jenkin's commit abdc746 and #26 not changing the SCM tag value it was not restored to HEAD in Jenkin's commit 54fec37. This PR corrects the SCM tag by restoring to HEAD in order to prepare for next development iteration.

Motivation and Context

Avoid any possible issues with future builds.

How Has This Been Tested?

  • mvn clean install, however, this doesn't test the change.

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.

Restore to HEAD to prepare for next development iteration.
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 requested a review from gschneider-r7 May 26, 2022 21:32
@gschneider-r7 gschneider-r7 merged commit cd67ab1 into rapid7:master May 26, 2022
@mkienow-r7 mkienow-r7 deleted the bug/correct-pom-scm-tag branch May 26, 2022 21:37
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