Skip to content

Add a Candidates#toString method to make debugging easier#1142

Merged
laeubi merged 1 commit into
eclipse-equinox:masterfrom
laeubi:candidates_to_string
Sep 6, 2025
Merged

Add a Candidates#toString method to make debugging easier#1142
laeubi merged 1 commit into
eclipse-equinox:masterfrom
laeubi:candidates_to_string

Conversation

@laeubi
Copy link
Copy Markdown
Member

@laeubi laeubi commented Sep 6, 2025

Currently debugging problems in the resolver involves a comparing different alternatives solutions. As the overall state could be quite big inspecting the state of the object itself can be quite challenging.

This now adds a new Candidates#toString method that allows for easy print out of the state of a Candidates object enriching it with some extracted data like if a requirement is optional or a substitution package and what a resource currently provides to others.

This is extracted from

as it becomes quite handy when looking at different candidate sets.

Currently debugging problems in the resolver involves a comparing
different alternatives solutions. As the overall state could be quite
big inspecting the state of the object itself can be quite challenging.

This now adds a new Candidates#toString method that allows for easy
print out of the state of a Candidates object enriching it with some
extracted data like if a requirement is optional or a substitution
package and what a resource currently provides to others.
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Sep 6, 2025

Test Results

  810 files  ±0    810 suites  ±0   1h 25m 6s ⏱️ - 3m 56s
2 221 tests ±0  2 172 ✅ ±0   49 💤 ±0  0 ❌ ±0 
6 677 runs  ±0  6 528 ✅ ±0  149 💤 ±0  0 ❌ ±0 

Results for commit 7bb80ba. ± Comparison against base commit 6da785c.

@laeubi laeubi merged commit 1132e94 into eclipse-equinox:master Sep 6, 2025
29 of 30 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