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.

Sign up for GitHub

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

Jump to bottom

Separate CDI Layer from Implementation #271

Closed
radcortez opened this issue Mar 31, 2020 · 0 comments · Fixed by #465
Closed

Separate CDI Layer from Implementation #271

radcortez opened this issue Mar 31, 2020 · 0 comments · Fixed by #465
Milestone
2.0.0

Comments

@radcortez
Copy link
Member

An initial thought is that the CDI layer would retain the current artifact name, so as not to break any downstream consumers, and then the pure Java implementation would be in a new artifact named {currentName]-core

The CDI artifact would then depend on the "-core" artifact for the bulk of the implementation and only contain the CDI specific pieces.
radcortez added a commit to radcortez/smallrye-config that referenced this issue Mar 31, 2020
@radcortez
Fixes smallrye#271 - Moved CDI related code to its own module.
630b848
radcortez added a commit to radcortez/smallrye-config that referenced this issue Mar 31, 2020
@radcortez
Fixes smallrye#271 - Moved CDI related code to its own module.
6b08b1e
@radcortez radcortez linked a pull request Apr 13, 2020 that will close this issue
Fixes #271 - Moved CDI related code to its own module. #272
Closed
@radcortez radcortez added this to the 1.8 milestone Apr 16, 2020
radcortez added a commit to radcortez/smallrye-config that referenced this issue Jun 15, 2020
@radcortez
Fixes smallrye#271 - Moved CDI related code to its own module.
71716b1
radcortez added a commit to radcortez/smallrye-config that referenced this issue Jun 15, 2020
@radcortez
Fixes smallrye#271 - Moved CDI related code to its own module.
9ba16a8
radcortez added a commit to radcortez/smallrye-config that referenced this issue Jun 16, 2020
@radcortez
Merge branch 'master' into smallrye#271-cdi-split
6443c5f
radcortez added a commit to radcortez/smallrye-config that referenced this issue Jun 17, 2020
@radcortez
Fixes smallrye#271 - Moved CDI related code to its own module.
8347b5b
radcortez added a commit to radcortez/smallrye-config that referenced this issue Jun 23, 2020
@radcortez
Fixes smallrye#271 - Moved CDI related code to its own module.
fb05875
radcortez added a commit to radcortez/smallrye-config that referenced this issue Jul 1, 2020
@radcortez
Fixes smallrye#271 - Moved CDI related code to its own module.
022afc2
radcortez added a commit to radcortez/smallrye-config that referenced this issue Jul 17, 2020
@radcortez
Merge branch 'master' into smallrye#271-cdi-split
97619b5
radcortez added a commit to radcortez/smallrye-config that referenced this issue Jul 17, 2020
@radcortez
Fixes smallrye#271 - Moved CDI related code to its own module.
74cdc23
radcortez added a commit to radcortez/smallrye-config that referenced this issue Jul 17, 2020
@radcortez
Fixes smallrye#271 - Moved CDI related code to its own module.
1f9c881
@radcortez radcortez mentioned this issue Dec 14, 2020
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
2.0.0
1 participant
@radcortez