Skip to content

Refactor gradle plugin so that tasks can be constructed independently#478

Merged
edward3h merged 8 commits intocasid:mainfrom
edward3h:gradle_tasks_take_2
Nov 21, 2025
Merged

Refactor gradle plugin so that tasks can be constructed independently#478
edward3h merged 8 commits intocasid:mainfrom
edward3h:gradle_tasks_take_2

Conversation

@edward3h
Copy link
Copy Markdown
Collaborator

Add independent properties to the task classes, and modify how they get "wired" with extension properties. New test case "jte-runtime-cp-test-gradle-newtask" demonstrates how a task can be created.

I also tried out changing Property<Path> to DirectoryProperty and Property<String[]> to ListProperty<String>, but I concluded this is not possible without breaking existing users of the plugin. Perhaps something to add if we make a new major version.

@edward3h
Copy link
Copy Markdown
Collaborator Author

I should have done a clean and rerun tests before I committed :)

@edward3h edward3h added the gradle Related to the Gradle plugin or use of Gradle. label Sep 8, 2025
@edward3h edward3h marked this pull request as draft September 8, 2025 16:07
@edward3h
Copy link
Copy Markdown
Collaborator Author

edward3h commented Sep 8, 2025

I pushed my implementation of a workaround, to make sure checks pass. This still needs some code comments to explain, and documentation updates.

@codecov
Copy link
Copy Markdown

codecov Bot commented Sep 8, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 91.17%. Comparing base (d40bb7c) to head (77755ae).
⚠️ Report is 9 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff            @@
##               main     #478   +/-   ##
=========================================
  Coverage     91.17%   91.17%           
  Complexity     1223     1223           
=========================================
  Files            76       76           
  Lines          3174     3174           
  Branches        492      492           
=========================================
  Hits           2894     2894           
  Misses          169      169           
  Partials        111      111           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@edward3h edward3h marked this pull request as ready for review November 15, 2025 18:56
@edward3h edward3h requested a review from casid November 15, 2025 18:56
@edward3h edward3h merged commit 9fc2ddd into casid:main Nov 21, 2025
15 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

gradle Related to the Gradle plugin or use of Gradle.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant