Skip to content
This repository has been archived by the owner on Oct 15, 2024. It is now read-only.

Documentation fix for elektraMetaArrayToKS #2327

Merged
merged 2 commits into from
Jan 23, 2019

Conversation

kodebach
Copy link
Member

Closes #2316

@markus2330
Copy link
Contributor

Thank you for fixing the docu!

"with value #1" is actually a deprecated way (it should be in meta data "array") but this PR is okay as it documents what it does.

@kodebach
Copy link
Member Author

kodebach commented Jan 23, 2019

"with value #1" is actually a deprecated way (it should be in meta data "array")

The function takes the array parent key as-is and appends it to the result. Modifying the resulting keys and adding meta-data onto meta-keys seems a bit unexpected to me.

Also the build server problems are definitely not caused by the documentation fixes made in this PR, so I think we can merge this regardless.

@markus2330 markus2330 merged commit 6dc8e61 into ElektraInitiative:master Jan 23, 2019
@markus2330
Copy link
Contributor

Thank you!

@kodebach kodebach deleted the meta-doc-fix branch August 25, 2019 11:53
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants