Skip to content

SCANGRADLE-293 mark SonarResolverTask as UntrackedTask#392

Merged
erwan-serandour merged 11 commits intomasterfrom
SCANGRADLE-293-c
Nov 13, 2025
Merged

SCANGRADLE-293 mark SonarResolverTask as UntrackedTask#392
erwan-serandour merged 11 commits intomasterfrom
SCANGRADLE-293-c

Conversation

@erwan-serandour
Copy link
Copy Markdown
Contributor

@erwan-serandour erwan-serandour commented Nov 11, 2025

SCANGRADLE-293

If tasks affect classpath gralde will detect that and throw an error relative to the SonarResolverTask, because SonarResolverTask depends on the classpath.

Two solution exits do not use any gradle caching mechanism with UntrackedTask or specify an explicit
relation (depends on, must run after, must run before, ... ) between the tasks.

@erwan-serandour erwan-serandour marked this pull request as draft November 11, 2025 16:33
@erwan-serandour erwan-serandour marked this pull request as ready for review November 11, 2025 16:44
@dorian-burihabwa-sonarsource
Copy link
Copy Markdown
Contributor

@erwan-serandour can you check the ticket number? It looks like you are working on different issue than SCANGRADLE-299

@erwan-serandour erwan-serandour changed the title SCANGRADLE-299 mark SonarResolverTask as UntrackedTask SCANGRADLE-293 mark SonarResolverTask as UntrackedTask Nov 12, 2025
@erwan-serandour erwan-serandour marked this pull request as draft November 12, 2025 08:25
@erwan-serandour erwan-serandour marked this pull request as ready for review November 12, 2025 10:00
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

It looks like we are on the right track. With this PR, we do 2 things:

  1. We mark the task as untracked
  2. We remove the Input and Output annotations
    Is there a way to make this solution work by simply doing the first change?

Comment thread integrationTests/src/test/java/org/sonarqube/gradle/GradleTest.java Outdated
Comment thread integrationTests/src/test/java/org/sonarqube/gradle/GradleTest.java Outdated
Comment thread integrationTests/src/test/java/org/sonarqube/gradle/GradleTest.java Outdated
Comment thread integrationTests/src/test/resources/java-gradle-classpath-dependency/.gitignore Outdated
Comment thread integrationTests/src/test/java/org/sonarqube/gradle/GradleTest.java Outdated
Comment thread src/main/java/org/sonarqube/gradle/SonarResolverTask.java Outdated
Comment thread src/main/java/org/sonarqube/gradle/SonarResolverTask.java Outdated
@sonarqube-next
Copy link
Copy Markdown

@erwan-serandour erwan-serandour merged commit 74aec4c into master Nov 13, 2025
38 of 39 checks passed
@erwan-serandour erwan-serandour deleted the SCANGRADLE-293-c branch November 13, 2025 14:02
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.

2 participants