Skip to content

Generalize the Capabilities class and move it to exported package#935

Open
laeubi wants to merge 1 commit intoeclipse-equinox:masterfrom
laeubi:make_cap_generic
Open

Generalize the Capabilities class and move it to exported package#935
laeubi wants to merge 1 commit intoeclipse-equinox:masterfrom
laeubi:make_cap_generic

Conversation

@laeubi
Copy link
Member

@laeubi laeubi commented May 3, 2025

Once in a while there is a demand to find providers for a requirement in a way that is not covered by the current wiring state, e.g. when traversing a repository or some other state of Resources. As indexing is hard to get done without access to some framework internals only using public API is not trivial. Equinox already contains a Capabilities class that supports such indexing using internal of the framework impl.

This makes the Capabilities class generic, so it can be used with any way of obtaining capabilities from a resource and move it to a public package.

See for example:

Once in a while there is a demand to find providers for a requirement in
a way that is not covered by the current wiring state, e.g. when
traversing a repository or some other state of Resources. As indexing is
hard to get done without access to some framework internals only using
public API is not trivial. Equinox already contains a Capabilities class
that supports such indexing using internal of the framework impl.

This makes the Capabilities class generic, so it can be used with any
way of obtaining capabilities from a resource and move it to a public
package.
@laeubi laeubi requested a review from tjwatson May 3, 2025 05:57
@github-actions
Copy link

github-actions bot commented May 3, 2025

Test Results

0 files   -   669  0 suites   - 669   0s ⏱️ - 1h 15m 12s
0 tests  - 2 224  0 ✅  - 2 177  0 💤  -  47  0 ❌ ±0 
0 runs   - 6 816  0 ✅  - 6 673  0 💤  - 143  0 ❌ ±0 

Results for commit 8044556. ± Comparison against base commit 586e30b.

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