Skip to content

Conversation

@edward3h
Copy link
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
Collaborator Author

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

@edward3h edward3h added the gradle label Sep 8, 2025
@edward3h edward3h marked this pull request as draft September 8, 2025 16:07
@edward3h
Copy link
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

codecov bot commented Sep 8, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 91.17%. Comparing base (e1803ae) to head (0e8f095).
⚠️ Report is 1 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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant