Skip to content

Conversation

@Artur-
Copy link
Member

@Artur- Artur- commented Dec 20, 2025

This is actually not needed as NodeTasks does lookup on the interface and that invokes the service loader on demand:

EndpointGeneratorTaskFactory endpointGeneratorTaskFactory = lookup.lookup(EndpointGeneratorTaskFactory.class);

@Artur- Artur- force-pushed the remove-task-runtime-dep branch from e0d9ee7 to 3b44ea1 Compare December 22, 2025 13:20
@Artur- Artur- force-pushed the remove-task-runtime-dep branch from 3b44ea1 to a75d274 Compare December 22, 2025 13:26
@github-actions
Copy link

github-actions bot commented Dec 22, 2025

Test Results

1 307 files  ±0  1 307 suites  ±0   1h 15m 5s ⏱️ -11s
9 273 tests ±0  9 205 ✅ ±0  68 💤 ±0  0 ❌ ±0 
9 736 runs  +6  9 660 ✅ +5  76 💤 +1  0 ❌ ±0 

Results for commit ca21358. ± Comparison against base commit 7641d50.

♻️ This comment has been updated with latest results.

@Artur- Artur- marked this pull request as ready for review January 9, 2026 09:50
@Artur- Artur- requested a review from caalador January 9, 2026 09:50
@sonarqubecloud
Copy link

sonarqubecloud bot commented Jan 9, 2026

@caalador caalador merged commit 3e790a8 into main Jan 9, 2026
30 of 31 checks passed
@caalador caalador deleted the remove-task-runtime-dep branch January 9, 2026 10:24
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.

4 participants