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 helidon-config-yaml-mp as a dependency in helidon-microprofile-config #4379

Merged
merged 2 commits into from
Jun 14, 2022

Conversation

klustria
Copy link
Member

@klustria klustria commented Jun 11, 2022

Fix for issue #4375:

  1. helidon-config-yaml-mp was recently decoupled from helidon-config-mp which is part of helidon-microprofile-config and as a consequence, is not a part of that config bundle anymore. This work was done to add it back and avoid any effects of incompatibility.
  2. Changes in tests/integration/config/hocon-mp are related to a minor clean up work.

klustria added 2 commits June 11, 2022 02:12
…nfig.

helidon-config-yaml-mp was recently decoupled from helidon-config-mp which is part of helidon-microprofile-config and as a consequence, is not a part of that config bundle anymore. This work was done to add it back and avoid any effects of incompatibility.
@klustria klustria requested a review from tomas-langer June 14, 2022 19:08
@klustria klustria merged commit aca9bf7 into helidon-io:master Jun 14, 2022
klustria added a commit to klustria/helidon that referenced this pull request Jun 15, 2022
…nfig. (helidon-io#4379)

helidon-config-yaml-mp was recently decoupled from helidon-config-mp which is part of helidon-microprofile-config and as a consequence, is not a part of that config bundle anymore. This work was done to add it back and avoid any effects of incompatibility.

(cherry picked from commit aca9bf7)
klustria added a commit that referenced this pull request Jun 16, 2022
…nfig. (#4379) (#4394)

helidon-config-yaml-mp was recently decoupled from helidon-config-mp which is part of helidon-microprofile-config and as a consequence, is not a part of that config bundle anymore. This work was done to add it back and avoid any effects of incompatibility.

(cherry picked from commit aca9bf7)
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.

2 participants