Skip to content

Conversation

h0tk3y
Copy link
Member

@h0tk3y h0tk3y commented Sep 18, 2025

  • Display error messages from DCL analysis in popups over code
  • Display the plugins contributing features and projec types
  • Add grouping to the model actions in the left pane

@h0tk3y h0tk3y requested a review from eskatos September 18, 2025 20:02
@h0tk3y h0tk3y self-assigned this Sep 18, 2025
Copy link
Member

@eskatos eskatos left a comment

Choose a reason for hiding this comment

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

Good stuff!

My comments are mostly on the Gradle version in use and the Wrapper validation failing.

distributionBase=GRADLE_USER_HOME
distributionPath=wrapper/dists
distributionUrl=https\://services.gradle.org/distributions-snapshots/gradle-9.2.0-branch-sigushkinDeclarativeComposability-20250818215312+0000-bin.zip
distributionUrl=https\://services.gradle.org/distributions-snapshots/gradle-9.2.0-20250903110058+0000-bin.zip
Copy link
Member

Choose a reason for hiding this comment

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

❓ I guess we could use 9.2.0-milestone-2 here

implementation("org.gradle:gradle-declarative-dsl-tooling-models:9.0-milestone-1")
implementation("org.gradle:gradle-declarative-dsl-core:9.2.0-milestone-1")
implementation("org.gradle:gradle-declarative-dsl-evaluator:9.2.0-milestone-1")
implementation("org.gradle:gradle-declarative-dsl-tooling-models:9.2.0-milestone-1")
Copy link
Member

Choose a reason for hiding this comment

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

❓ I guess we could use 9.2.0-milestone-2 here

Copy link
Member

Choose a reason for hiding this comment

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

❌ Wrapper JAR validation check failed. I don't know where this one is coming from but using the 9.2.0-milestone-2 should work fine.

Copy link
Member Author

Choose a reason for hiding this comment

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

Unfortunately, 9.2.0-milestone-2 fails the check just as well.

GetKotlinBaseDslScriptModel(),
GetResilientGradleBuild(),
GetResilientKotlinDslScriptsModel(),
val modelActionGroups = listOf(
Copy link
Member

Choose a reason for hiding this comment

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

👍 Good call adding some structure here

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