Skip to content

Extract ClasspathContainerState from ClasspathComputer#1905

Merged
laeubi merged 1 commit intoeclipse-pde:masterfrom
laeubi:extract_classpath_container_state
Jul 28, 2025
Merged

Extract ClasspathContainerState from ClasspathComputer#1905
laeubi merged 1 commit intoeclipse-pde:masterfrom
laeubi:extract_classpath_container_state

Conversation

@laeubi
Copy link
Copy Markdown
Contributor

@laeubi laeubi commented Jul 28, 2025

With the enhanced storage/save of the classpath container state now the ClasspathComputer becomes more and more complex and contains too much different tasks.

This extracts from the ClasspathComputer the parts that handle the state into new class ClasspathContainerState to make it more maintainable.

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Jul 28, 2025

Test Results

   765 files  ±0     765 suites  ±0   52m 15s ⏱️ - 2m 35s
 3 611 tests ±0   3 557 ✅ ±0   54 💤 ±0  0 ❌ ±0 
10 834 runs  ±0  10 671 ✅ ±0  163 💤 ±0  0 ❌ ±0 

Results for commit abe0c06. ± Comparison against base commit 4c4ad88.

♻️ This comment has been updated with latest results.

@laeubi laeubi force-pushed the extract_classpath_container_state branch from 0b456c8 to 074ff4c Compare July 28, 2025 08:45
With the enhanced storage/save of the classpath container state now the
ClasspathComputer becomes more and more complex and contains too much
different tasks.

This extracts from the ClasspathComputer the parts that handle the state
into new class ClasspathContainerState to make it more maintainable.
@laeubi laeubi force-pushed the extract_classpath_container_state branch from 074ff4c to abe0c06 Compare July 28, 2025 08:54
@laeubi laeubi merged commit bed58e3 into eclipse-pde:master Jul 28, 2025
19 checks passed
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.

1 participant