-
Notifications
You must be signed in to change notification settings - Fork 18
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Rename release profile #133
Conversation
As we dumped nx2 staging plugin, rename the profile ID to something more logical: "sisu-release".
@@ -456,7 +456,7 @@ | |||
|
|||
<profiles> | |||
<profile> | |||
<id>sonatype-oss-release</id> | |||
<id>sisu-release</id> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why not the default profile name release
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
As it may conflict with out of the box profile, and as you see, release plugin explicitly disables it.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Also, the release profile used by project was not "default" anyway, this PR does not change in this respect, just shortens its name (and aligns it, with other "release profiles" like apache-release
or takari-release
etc).
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I still don't get why we need a project-specific prefix. I like to have the same profile name across all projects, therefore I usually pick release
. But as it always has been used with some prefix I am fine with keeping it.
|
As we dumped nx2 staging plugin, rename the profile ID to something more logical: "sisu-release".