Skip to content

Add the ability to use bash variables when setting keys#10

Open
fattredd wants to merge 2 commits intodamphat:masterfrom
fattredd:feature/multiWord
Open

Add the ability to use bash variables when setting keys#10
fattredd wants to merge 2 commits intodamphat:masterfrom
fattredd:feature/multiWord

Conversation

@fattredd
Copy link
Contributor

Now we can run

$ test="a b c"
$ kvset key 1 2 3 4 $test
$ kvget key
> 1 2 3 4 a b c

This is very helpful when using kv-bash for storing arrays.

@damphat
Copy link
Owner

damphat commented Apr 25, 2022

Good idea Jack!

Although your idea is good, but it does change the API signature.

Let me think more about it before merging because it might effect someone else who used the API.

@fattredd
Copy link
Contributor Author

That's fair. Good call

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