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

Issue #385: Allow import of type systems published through SPI #394

Merged

Conversation

reckart
Copy link
Member

@reckart reckart commented Sep 26, 2024

What's in the PR

  • Enhance the Import_impl to look up descriptors via the type system description SPI
  • Bit of cleaning up in related classes

How to test manually

  • No specific test procedure

Automatic testing

  • PR adds/updates unit tests

Documentation

  • PR adds/updates documentation

Organizational

  • PR adds/updates dependencies.
    Only dependencies under approved licenses are allowed. LICENSE and NOTICE files in the respective modules where dependencies have been added as well as in the project root have been updated.

- Enhance the Import_impl to look up descriptors via the type system description SPI
- Bit of cleaning up in related classes
@reckart reckart added the ⭐️ Enhancement Improvement or new feature for users label Sep 26, 2024
@reckart reckart added this to the 3.6.0 milestone Sep 26, 2024
@reckart reckart self-assigned this Sep 26, 2024
- Consolidate the different provider interfaces into a TypeSystemProvider interface and provide an abstract base implementation with caching
- Deprecate current SPI interfaces in favor of the new combined interface
- Update documentation
- Consider the new combined interface in locations where we would previously only have considered the current interfaces
@reckart reckart merged commit 4f0776c into main Sep 30, 2024
3 checks passed
@reckart reckart deleted the feature/385-Allow-import-of-type-systems-published-through-SPI branch September 30, 2024 10:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
⭐️ Enhancement Improvement or new feature for users
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant