Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Add support for customProjectName property to override default project name #24

Merged
merged 2 commits into from
Feb 1, 2018
Merged

Conversation

jrjohnson87
Copy link
Contributor

@jrjohnson87 jrjohnson87 commented Nov 30, 2017

In our use case, the default project name value is making it difficult to distinguish various projects across different teams, especially PR builds. This allows a property to be specified that sets the project name value to something other than build.getParent().getName()

…override build.getParent().getName()

* Add support for specifying project name

* Fixed compile issue

* Fix unit test

* Fix test

* Debug tests

* Add unit tests
@jrjohnson87 jrjohnson87 reopened this Dec 12, 2017
@timja
Copy link
Member

timja commented Jan 27, 2018

@asimell would you be able to review this please?

@asimell
Copy link
Contributor

asimell commented Jan 27, 2018

@timja-kainos Yes, I was actually finally able to review this a couple of hours ago and the code looked good, but unfortunately merging it will go to next week due to personal scheduling issues.

@asimell
Copy link
Contributor

asimell commented Jan 31, 2018

@jrjohnson87 Currently, there is no method to add a custom project name for non-pipeline jobs. Could you add the proper configuration for the config.jelly file?

@jrjohnson87
Copy link
Contributor Author

jrjohnson87 commented Jan 31, 2018

@asimell Updated to add this. Let me know if the help wording is acceptable or needs re-phrasing.

@asimell asimell merged commit 0b95ddc into jenkinsci:development Feb 1, 2018
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants