Skip to content

Add util func to concat map values#285

Merged
osinstom merged 11 commits intomainfrom
rranjan3/util
Jun 24, 2025
Merged

Add util func to concat map values#285
osinstom merged 11 commits intomainfrom
rranjan3/util

Conversation

@rranjan3
Copy link
Copy Markdown
Contributor

@rranjan3 rranjan3 commented Jun 23, 2025

Description

Add ConcatMapValuesSorted func to inventory util. It takes a map of string keys and values, sorts the keys alphabetically,
concatenates the corresponding non-empty values using the specified delimiter.

Fixes # (issue)

Any Newly Introduced Dependencies

Please describe any newly introduced 3rd party dependencies in this change. List their name, license information and how they are used in the project.

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

Checklist:

  • I agree to use the APACHE-2.0 license for my code changes
  • I have not introduced any 3rd party dependency changes
  • I have performed a self-review of my code

Signed-off-by: Rajeev Ranjan <rajeev2.ranjan@intel.com>
Copy link
Copy Markdown
Contributor

@pierventre pierventre left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

can you check if there is a collection/map pkg already ?

Signed-off-by: Rajeev Ranjan <rajeev2.ranjan@intel.com>
@rranjan3
Copy link
Copy Markdown
Contributor Author

can you check if there is a collection/map pkg already ?

Moved to collections.

Signed-off-by: Rajeev Ranjan <rajeev2.ranjan@intel.com>
@rranjan3
Copy link
Copy Markdown
Contributor Author

Signed-off-by: Rajeev Ranjan <rajeev2.ranjan@intel.com>
rranjan3 added 7 commits June 24, 2025 13:29
Signed-off-by: Rajeev Ranjan <rajeev2.ranjan@intel.com>
Signed-off-by: Rajeev Ranjan <rajeev2.ranjan@intel.com>
Signed-off-by: Rajeev Ranjan <rajeev2.ranjan@intel.com>
Signed-off-by: Rajeev Ranjan <rajeev2.ranjan@intel.com>
Signed-off-by: Rajeev Ranjan <rajeev2.ranjan@intel.com>
@osinstom osinstom merged commit 59a77f2 into main Jun 24, 2025
76 checks passed
@osinstom osinstom deleted the rranjan3/util branch June 24, 2025 16:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants