Where communities thrive


  • Join over 1.5M+ people
  • Join over 100K+ communities
  • Free without limits
  • Create your own community
People
Repo info
Activity
    Emily Jiang
    @Emily-Jiang
    it is official
    We have a final build
    Roberto Cortez
    @radcortez
    congratulations!!
    Emily Jiang
    @Emily-Jiang
    time to celebrate....
    Roberto Cortez
    @radcortez
    well not yet, lets pass the vote first :)
    Emily Jiang
    @Emily-Jiang
    celebration in stages...
    let me create an issue for the vote... please create an issue for CCR with MP Config 2.0
    Emily Jiang
    @Emily-Jiang
    microprofile/microprofile-wg#8 can you review
    Emily Jiang
    @Emily-Jiang
    no worries with the review
    microprofile/microprofile-wg#9 the one to be used for final voting... can you create a CCR
    @radcortez
    Roberto Cortez
    @radcortez
    y sec
    Emily Jiang
    @Emily-Jiang
    thanks
    Roberto Cortez
    @radcortez
    the staging repo link is not correct: https://oss.sonatype.org/content/repositories/orgeclipsemicroprofile-1357/org/eclipse/microprofile/config/
    is 1375 and not 1357 :)
    Emily Jiang
    @Emily-Jiang
    That is an explaination from the template
    Roberto Cortez
    @radcortez
    ah sorry
    Emily Jiang
    @Emily-Jiang
    np
    Roberto Cortez
    @radcortez
    it is confusing because it is a link so the instict is to click on it
    we may need to drop that piece
    Emily Jiang
    @Emily-Jiang
    maybe change 1357 to xxxx
    Roberto Cortez
    @radcortez
    and drop the link
    Emily Jiang
    @Emily-Jiang
    I think the link helps with someone who tries to figure what links to attach
    Roberto Cortez
    @radcortez
    ok
    Roberto Cortez
    @radcortez
    eclipse/microprofile-config#656
    I’ve reused the same issue
    and just changed the links
    Emily Jiang
    @Emily-Jiang
    awesome. thanks
    I have addressed your comments
    microprofile/microprofile-wg#10
    to make the link not showing as a link
    you are quick. Thanks!
    Roberto Cortez
    @radcortez
    :D
    Emily Jiang
    @Emily-Jiang
    By the way, I have asked Kevin to mentor MP Config release. He will send an email to microprofile-wg for voting. I'll send email to the other parties and then give him a go ahead.
    Roberto Cortez
    @radcortez
    Ok!
    Emily Jiang
    @Emily-Jiang
    thanks
    Duncan
    @dabloem
    Hi all, I was wondering if there is already something to bring mp-config to spring-boot?
    So, developers can use @ConfigProperty instead of @Value ?
    Jan Bernitt
    @jbee
    isn't MP config dependent on CDI which you do not have in a spring environment?
    Roberto Cortez
    @radcortez
    Yes, for @NonBinding
    spring supports @Inject, not sure now it deals with @NonBinding
    Jan Bernitt
    @jbee
    surely you can make the annotation work in some way on top of spring concepts but it might need some adaptors or even replacement of classes I would guess
    Duncan
    @dabloem
    Well, I did some proof of concept. And it works
    however, I am not a spring guru
    Roberto Cortez
    @radcortez
    which MP Config implementation are you using?
    Duncan
    @dabloem
    I implemented a BeanPostProcessor which is looking for ConfigProperty on fields, and if found it adds a FieldCallBack, which does the lookup via ConfigProvider.getConfig()
    Roberto Cortez
    @radcortez
    ah great
    Duncan
    @dabloem
    Smallrye and Geronimo both work
    Roberto Cortez
    @radcortez
    unfortunately for a single annotation you do required to have the CDI API jar in your CP
    Duncan
    @dabloem
    But because it was not that hard, I was just wondering if this already exists, and probably a bit more mature
    Roberto Cortez
    @radcortez
    not that I’m aware, because no one ever asked for it. usually spring developers stick to what they already have