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

Set modifyindex flag to use the documented name #13

Merged
merged 1 commit into from
Jan 7, 2016

Conversation

agy
Copy link
Contributor

@agy agy commented Jan 7, 2016

The kv-write command is documented as accepting the --modifyindex flag, but I received an error when attempting to use it.

./consul-cli kv-write --modifyindex=0 key value
unknown flag: --modifyindex

The source expects --cas instead, which contradicts the documentation. I modified kv-write to use the documented flag.

ChrisAubuchon added a commit that referenced this pull request Jan 7, 2016
Set `modifyindex` flag to use the documented name
@ChrisAubuchon ChrisAubuchon merged commit 626195e into mantl:master Jan 7, 2016
@ChrisAubuchon
Copy link
Contributor

Ack! Good catch! Thanks for the PR

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.

2 participants