Skip to content
This repository has been archived by the owner on Aug 3, 2023. It is now read-only.

add --preview to kv:key, kv:bulk + kv:namespace #1360

Merged
merged 6 commits into from
Jun 10, 2020

Conversation

EverlastingBugstopper
Copy link
Contributor

@EverlastingBugstopper EverlastingBugstopper commented Jun 3, 2020

lets do it up

@EverlastingBugstopper EverlastingBugstopper requested a review from a team as a code owner June 3, 2020 21:38
@EverlastingBugstopper EverlastingBugstopper changed the title wip [wip] add --preview to kv:key and kv:bulk Jun 3, 2020
@ashleymichal
Copy link
Contributor

it's probably colliding with the other preview arg.

@EverlastingBugstopper EverlastingBugstopper changed the title [wip] add --preview to kv:key and kv:bulk add --preview to kv:key and kv:bulk Jun 3, 2020
@EverlastingBugstopper
Copy link
Contributor Author

i figured it out

@EverlastingBugstopper
Copy link
Contributor Author

@ashleymichal ur refactor made this so easy thank u so much

@ashleymichal
Copy link
Contributor

are you adding for kv:namespace in a separate pr?

@EverlastingBugstopper
Copy link
Contributor Author

wasn't planning on doing for kv:namespace. what would the --preview flag do for kv:namespace?

@ashleymichal
Copy link
Contributor

wasn't planning on doing for kv:namespace. what would the --preview flag do for kv:namespace?

i think it would only change the messaging around what to put in your toml.

@EverlastingBugstopper EverlastingBugstopper changed the title add --preview to kv:key and kv:bulk add --preview to kv:key, kv:bulk + kv:namespace Jun 8, 2020
@EverlastingBugstopper EverlastingBugstopper force-pushed the avery/kv-preview branch 2 times, most recently from 8a9f90e to 803db3e Compare June 10, 2020 17:11
@EverlastingBugstopper EverlastingBugstopper merged commit 16a14b1 into avery/sneaky-snakey-preview Jun 10, 2020
@ispivey ispivey added this to the 1.10.0 milestone Jun 11, 2020
@EverlastingBugstopper EverlastingBugstopper deleted the avery/kv-preview branch August 3, 2020 17:08
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants