Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add Extender that allows bundles to declare a transformer resource #732

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

laeubi
Copy link
Member

@laeubi laeubi commented Dec 21, 2024

Currently it is rather inconvenient to make a transformer list available as it requires to register an URL as service and the service itself also needs to be registered with some properties.

This now adds an bundle extender that scans for a header 'Equinox-Transformer' where the value is pointing to a resource in the bundle (optionally specify the transformer type). If such bundle is found the resource is looked up and registered as a service with the required properties so it can be discovered by the transformer.

Contributes to

Requires:

@laeubi laeubi force-pushed the transformer_extender branch from 88733ba to 87d92a4 Compare December 21, 2024 08:09
Currently it is rather inconvenient to make a transformer list available
as it requires to register an URL as service and the service itself also
needs to be registered with some properties.

This now adds an bundle extender that scans for a header
'Equinox-Transformer' where the value is pointing to a resource in the
bundle (optionally specify the transformer type). If such bundle is
found the resource is looked up and registered as a service with the
required properties so it can be discovered by the transformer.
@laeubi laeubi force-pushed the transformer_extender branch from 87d92a4 to 1daacf5 Compare December 21, 2024 08:25
Copy link

Test Results

0 files   -   663  0 suites   - 663   0s ⏱️ - 1h 13m 34s
0 tests  - 2 211  0 ✅  - 2 164  0 💤  -  47  0 ❌ ±0 
0 runs   - 6 777  0 ✅  - 6 634  0 💤  - 143  0 ❌ ±0 

Results for commit 1daacf5. ± Comparison against base commit 5810f68.

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