Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

[FAB-17166] Gate UT, IT and DocBuild #337

Merged
merged 1 commit into from
Dec 2, 2019
Merged

[FAB-17166] Gate UT, IT and DocBuild #337

merged 1 commit into from
Dec 2, 2019

Conversation

lindluni
Copy link
Contributor

@lindluni lindluni commented Dec 1, 2019

Adds a gate behind VerifyBuild that will tell the other jobs which of them should run depending on which directories in the source tree have been updated

Signed-off-by: Brett Logan [email protected]

Type of change

  • Improvement (improvement to code, performance, etc)

Description

This change introduces logic to check if the current commit set has made changes to just the /docs tree and tells CI to only run DocBuild if this is the case. This makes sure DocBuild doesn't have to wait on UT and IT to run. Otherwise it runs all tests

Adds a gate behind VerifyBuild that will tell
the other jobs which of them should run depending
on which directories in the source tree have
been updated

Signed-off-by: Brett Logan <[email protected]>
@lindluni lindluni requested a review from guoger December 2, 2019 03:51
Copy link
Contributor

@guoger guoger left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. CI creates a merge commit prior to running the script, which makes HEAD^..HEAD work as expected (against changes in all commits)

@kchristidis kchristidis merged commit 94c3a09 into hyperledger:master Dec 2, 2019
@lindluni lindluni deleted the azp_gate branch January 23, 2020 15:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants