Skip to content

Remove setting of autoReleaseAfterClose.#20

Merged
jradcliff merged 1 commit intomasterfrom
deployautorel
Aug 22, 2018
Merged

Remove setting of autoReleaseAfterClose.#20
jradcliff merged 1 commit intomasterfrom
deployautorel

Conversation

@jradcliff
Copy link
Copy Markdown
Member

The previous attempt to use a build property for this setting did not
work as expected. The nexus-staging-maven-plugin ignored the system
property passed on the command line.

In addition, it turns out that this property is not necessary for two
reasons:

  1. The autoReleaseAfterClose property defaults to false. See:
    https://help.sonatype.com/repomanager2/staging-releases/configuring-your-project-for-deployment#ConfiguringYourProjectforDeployment-DeploymentwiththeNexusStagingMavenPlugin
  2. The autoReleaseAfterClose property can be passed via the command
    line already, so there's no point in introducing an intermediate
    build property that mirros the existing property. See:
    https://github.com/sonatype/nexus-maven-plugins/tree/master/staging/maven-plugin#plugin-flags

[deployautorel]

The previous attempt to use a build property for this setting did not
work as expected. The `nexus-staging-maven-plugin` ignored the system
property passed on the command line.

In addition, it turns out that this property is not necessary for two
reasons:

1.  The `autoReleaseAfterClose` property defaults to `false`. See:
    https://help.sonatype.com/repomanager2/staging-releases/configuring-your-project-for-deployment#ConfiguringYourProjectforDeployment-DeploymentwiththeNexusStagingMavenPlugin
2.  The `autoReleaseAfterClose` property can be passed via the command
    line already, so there's no point in introducing an intermediate
    build property that mirros the existing property. See:
    https://github.com/sonatype/nexus-maven-plugins/tree/master/staging/maven-plugin#plugin-flags

[deployautorel]
@jradcliff jradcliff requested a review from AnashOommen August 21, 2018 13:39
@jradcliff jradcliff merged commit b608045 into master Aug 22, 2018
@jradcliff jradcliff deleted the deployautorel branch August 22, 2018 13:23
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.

2 participants