-
Notifications
You must be signed in to change notification settings - Fork 13
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
yaml list entries are shrinked to last one only #12
Comments
bumped with this yesterday, culprit seems to be here: https://github.com/ozimov/yaml-properties-maven-plugin/blob/master/src/main/java/org/codehaus/mojo/properties/YamlToPropertiesConverter.java#L80, can be replaced by something like:
|
Hi Juan Pablo,
Can you describe the issue inside a GitHub issue?
Best
Roberto
…On Thu, Feb 10, 2022 at 9:38 AM Juan Pablo Santos Rodríguez < ***@***.***> wrote:
bumped with this yesterday, culprit seems to be here:
https://github.com/ozimov/yaml-properties-maven-plugin/blob/master/src/main/java/org/codehaus/mojo/properties/YamlToPropertiesConverter.java#L80,
can be replaced by something like:
String val = value == null ? "" : value;
if( dataMap.get( key ) != null ) {
dataMap.put( key, dataMap.get( key ) + "," + val );
} else {
dataMap.put( key, val );
}
—
Reply to this email directly, view it on GitHub
<#12 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAWZPCQ3KMZZXMQUFSLKY7TU2N2PJANCNFSM4XMY7CEA>
.
Triage notifications on the go with GitHub Mobile for iOS
<https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675>
or Android
<https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>.
You are receiving this because you are subscribed to this thread.Message
ID: ***@***.***>
--
Eng. Roberto Trunfio, PhD
Lead Software Engineer
Data Team
Appway AG
|
Hi Roberto, The issue is the same described by @raleik-pl, hence the comment here. As the plugin doesn't seem to be in active development, instead of sending a PR, I just described where the bug is originated and how to fix it, in case anyone wants to fork or patch it locally. If you think it would be useful, I can send a PR, so a new release can be made. Best regards, |
If the yaml file looks like this:
options:
- one
- two
- three
then ${options} property is equeal to the last list item "three"
The text was updated successfully, but these errors were encountered: