I'm sorry, but I disagree with this change. "ChangeRequestSCMHead" should hold the pertinent SCM information needed to build the the change and I don't see the title as part of that.
For example, the title can change without the meaningful SCM information changing. Should the title be included in `equals()` comparisons? If the title changes, should a new build be triggered?
I understand it would be convenient to have this information, and I even support the Title of Change requests being added to some shared interface. However, there needs to be some additional discussion of other possible places to add this information and possible effects and changes in behavior.
I'm sorry. I want to support you in making improvements to this part of the API, but I'm have concerns about this improvement as it stands.
I'm sorry, but I disagree with this change. "ChangeRequestSCMHead" should hold the pertinent SCM information needed to build the the change and I don't see the title as part of that.
For example, the title can change without the meaningful SCM information changing. Should the title be included in `equals()` comparisons? If the title changes, should a new build be triggered?
I understand it would be convenient to have this information, and I even support the Title of Change requests being added to some shared interface. However, there needs to be some additional discussion of other possible places to add this information and possible effects and changes in behavior.
I'm sorry. I want to support you in making improvements to this part of the API, but I'm have concerns about this improvement as it stands.