-
Notifications
You must be signed in to change notification settings - Fork 35
[SHIPKIT-513] Automated commit message should have CI build ID #682
Conversation
@mockitoguy Some problem with testDownstream:testMockitoMockito stage. I'll reopen after I'll resolve them |
@mockito/shipkit-developers seems to be connected to #680 |
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 don't think this works quite yet, please check out my feedback.
t.setCommitMessage(System.getenv("TRAVIS_COMMIT_MESSAGE")); | ||
t.setPullRequest(isPullRequest); | ||
} | ||
project.getTasks().withType(ReleaseNeededTask.class, t -> { |
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.
We want to set the commit message on the GitCommitTask. We don't want to generate commit message for ReleaseNeededTask because this task should work on verbatim commit message supplied by Travis. ReleaseNeededTask is not making commits :)
Check out the initial design of this feature at #514
When you work on this feature, I suggest to test it this way:
./gradlew performRelease -PdryRun
Above will get you a local commit with the message. Please ensure that this commit has the stuff we need.
Thank you for working on this!!!
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.
Sure, I created PR to confirm my assumptions were right (which were wrong, but now I got feedback :)).
I'll add necessary changes.
e87ff42
to
0abad20
Compare
Solving issue #513