Skip to content
This repository has been archived by the owner on Jan 18, 2021. It is now read-only.

[SHIPKIT-513] Automated commit message should have CI build ID #682

Closed
wants to merge 7 commits into from

Conversation

ArturSkowronski
Copy link
Member

@ArturSkowronski ArturSkowronski commented Mar 26, 2018

Solving issue #513

@ArturSkowronski
Copy link
Member Author

@mockitoguy Some problem with testDownstream:testMockitoMockito stage. I'll reopen after I'll resolve them

@ArturSkowronski
Copy link
Member Author

@mockito/shipkit-developers seems to be connected to #680

Copy link
Member

@mockitoguy mockitoguy left a 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 -> {
Copy link
Member

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!!!

Copy link
Member Author

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.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants