-
Notifications
You must be signed in to change notification settings - Fork 6k
Verify Samples in Build #10031
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
Verify Samples in Build #10031
Conversation
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.
Nice, @marcusdacoregio! I've left some feedback inline.
buildSrc/src/main/groovy/io/spring/gradle/RemoteProjectTask.groovy
Outdated
Show resolved
Hide resolved
buildSrc/src/main/groovy/io/spring/gradle/RemoteProjectTask.groovy
Outdated
Show resolved
Hide resolved
f82aa0e
to
3ca272f
Compare
I've updated the PR with your suggestions @jzheaux. Can you review it again? Thanks! |
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.
Thanks, @marcusdacoregio! I've left some feedback inline.
buildSrc/src/main/groovy/io/spring/gradle/convention/VerifySamplesPlugin.groovy
Outdated
Show resolved
Hide resolved
Thanks for the review @jzheaux, I've updated the PR based on your suggestions. |
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.
Nice, @marcusdacoregio! I've left my remaining feedback inline. Once those are addressed feel free to merge.
64a0e11
to
4878239
Compare
Creates a new Gradle task named
checkSamples
which clones and performs a./gradlew check
on the Samples project.Adds a new job in the CI workflow which runs the new
checkSamples
task.Closes gh-9846