[JENKINS-63102] Added two new Project Naming Strategies for "Contextual Project Path" and "Simple Project Path" #181
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Added options for "Contextual" and "Simple" as per README and other documentation.
robross0606
changed the title
robross0606
commented
Mar 11, 2022
| @@ -93,7 +95,7 @@ | |||
| private String credentialsId; | |||
|
|
|||
| /** | |||
| * The behavioural traits to apply. | |||
There was a problem hiding this comment.
Minor spelling fixes.
There was a problem hiding this comment.
Sorry for all the unintended spelling "fixes". My Eclipse spelling plugin is set to US English and it flagged a whole bunch of corrections based on US English. I can revert these if you like.
robross0606
force-pushed
the
JENKINS-59759_additional_naming_strategies
branch
from
15c96f7 to
cc2f9db
Compare
jetersen
requested changes
Apr 18, 2022
src/main/java/io/jenkins/plugins/gitlabbranchsource/ProjectNamingStrategyTrait.java
Outdated
Show resolved
Hide resolved
...ources/io/jenkins/plugins/gitlabbranchsource/ProjectNamingStrategyTrait/help-strategyId.html
Outdated
Show resolved
Hide resolved
|
Please resolve the conflict :) |
…uctor` and `@DataBoundSetter`. Also updated help details for naming strategy to clarify use cases.
robross0606
force-pushed
the
JENKINS-59759_additional_naming_strategies
branch
from
8a84939 to
a7a046e
Compare
robross0606
changed the title
jetersen
approved these changes
Apr 18, 2022
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Added options for "Contextual" and "Simple" as per README and other
documentation.
This addressed the open JENKINS-63102. Details can be found on that ticket. This adds two additional options for Project Naming Strategy:
This has been tested via
mvn hpi:run: