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

Microprofile Config support for Java SE Bootstrap API #3943

Open
wants to merge 4 commits into
base: JAXRS_2_2
Choose a base branch
from

Conversation

mkarg
Copy link
Member

@mkarg mkarg commented Sep 23, 2018

This PR dependens on Support for JAX-RS 2.2 Java SE Bootstrapping API (PR #3839)

This PR adds support for Microprofile Config API to the Support for Java SE Bootstrapping (implemented by PR #3839).

@mkarg mkarg force-pushed the javase-microprofile branch 2 times, most recently from bffec01 to eeb5e1a Compare October 14, 2018 12:54
@mkarg mkarg changed the title WIP: Microprofile Config configuration provider for Java SE Bootstrap API Microprofile Config support for Java SE Bootstrap API Oct 14, 2018
@mkarg
Copy link
Member Author

mkarg commented Oct 14, 2018

@jansupol Review please. :-)

@mkarg mkarg force-pushed the javase-microprofile branch 3 times, most recently from c1d2c28 to 710a677 Compare February 23, 2019 18:13
@mkarg mkarg force-pushed the javase-microprofile branch 2 times, most recently from 9933a6a to cd74e6e Compare March 28, 2019 21:21
@jansupol
Copy link
Contributor

@mkarg There is currently some discussion about a configuration in Jersey. Optimistically, I hope there would be some proposal ready in a week or two. I think it makes sense to align this PR with that config (if applicable), which is personally holding me back for this PR. I hope you will like the proposal then.

@mkarg
Copy link
Member Author

mkarg commented Mar 30, 2019

Can you point me to the Github PR / issue that you mean?

@mkarg
Copy link
Member Author

mkarg commented Apr 12, 2019

@jansupol Two weeks is over, so I'd like to start with reworking this PR. Can you please point me to the new configuration API that I shall use in this PR favor of my own? Thanks. :-)

@jansupol
Copy link
Contributor

@mkarg I thought we would have been faster, currently, we have the following idea: #4110

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.

None yet

2 participants