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

Collect failed tasks and report as TeamCity build problems #21352

Merged
merged 1 commit into from
Aug 12, 2022

Conversation

blindpirate
Copy link
Collaborator

@blindpirate blindpirate commented Jul 26, 2022

In the past, if code quality tasks (checkstyle/codenarc/ktlint) failed, it's a bit hard for developers to find out the problems. This PR adds a build service to collect the failed tasks and report TeamCity build problems, which can be clicked directly Example:

https://builds.gradle.org/buildConfiguration/Gradle_Master_Check_SanityCheck/54874906?buildTab=overview&expandBuildProblemsSection=true&hideProblemsFromDependencies=false&hideTestsFromDependencies=false&expandBuildChangesSection=true

image

image

Because the new code shares a lot with previous cache-miss-monitor, a common method is extracted.

@blindpirate blindpirate requested a review from wolfs July 26, 2022 05:24
@blindpirate blindpirate added the @dev-productivity Issue owned by developer-productivity label Jul 26, 2022
@blindpirate blindpirate self-assigned this Jul 26, 2022
@blindpirate blindpirate added this to the 7.6 RC1 milestone Jul 26, 2022
@blindpirate blindpirate force-pushed the blindpirate/capture-failed-tasks branch 5 times, most recently from 14f9517 to 2052115 Compare August 11, 2022 06:57
@blindpirate blindpirate changed the title Collect failed tasks and report TeamCity build problems Collect failed tasks and report as TeamCity build problems Aug 11, 2022
@blindpirate blindpirate marked this pull request as ready for review August 11, 2022 07:01
In the past, if code quality tasks (checkstyle/codenarc/ktlint) failed,
it's a bit hard for developers to find out the problems.
This PR adds a build service to collect the failed tasks and
report TeamCity build problems, which can be clicked directly.
@blindpirate blindpirate force-pushed the blindpirate/capture-failed-tasks branch from 2052115 to 7e5e5b7 Compare August 11, 2022 07:01
@blindpirate
Copy link
Collaborator Author

@bot-gradle test this

@bot-gradle
Copy link
Collaborator

OK, I've already triggered the following builds for you:

@blindpirate blindpirate modified the milestones: 7.6 RC1, 8.0 RC1 Aug 12, 2022
@blindpirate
Copy link
Collaborator Author

Let's try this to see if people like it.

@bot-gradle test and merge

@gradle gradle deleted a comment from blindpirate Aug 12, 2022
@bot-gradle
Copy link
Collaborator

OK, I've already triggered a build for you.

@bot-gradle bot-gradle merged commit 18642c7 into master Aug 12, 2022
@ghost ghost mentioned this pull request Aug 12, 2022
@blindpirate blindpirate deleted the blindpirate/capture-failed-tasks branch August 12, 2022 05:08
bot-gradle added a commit that referenced this pull request Aug 19, 2022
Cherry-pick #21352 to release

Co-authored-by: Bo Zhang <bo@gradle.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
@dev-productivity Issue owned by developer-productivity
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants