Navigation Menu

Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

cmd/flux-kvs: Hide watch subcommand #1891

Merged
merged 1 commit into from Dec 21, 2018
Merged

Conversation

chu11
Copy link
Member

@chu11 chu11 commented Dec 21, 2018

The old watch interface is being deprecated, so similarly hide
the option from flux-kvs help output and remove documentation
of it from the manpage.

Fixes #1888

@codecov-io
Copy link

Codecov Report

Merging #1891 into master will increase coverage by 0.02%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master    #1891      +/-   ##
==========================================
+ Coverage   80.09%   80.12%   +0.02%     
==========================================
  Files         196      196              
  Lines       35001    35002       +1     
==========================================
+ Hits        28034    28045      +11     
+ Misses       6967     6957      -10
Impacted Files Coverage Δ
src/cmd/flux-kvs.c 83.18% <0%> (-0.1%) ⬇️
src/common/libutil/veb.c 98.28% <0%> (-0.58%) ⬇️
src/modules/kvs/kvs.c 66.71% <0%> (+0.14%) ⬆️
src/modules/connector-local/local.c 74.81% <0%> (+1.03%) ⬆️
src/modules/barrier/barrier.c 78.62% <0%> (+2.06%) ⬆️

@chu11
Copy link
Member Author

chu11 commented Dec 21, 2018

hehe 0% of diff coverage ... b/c it's just a change to help output

The old watch interface is being deprecated, so similarly hide
the option from flux-kvs help output and remove documentation
of it from the manpage.

Fixes flux-framework#1888
@chu11
Copy link
Member Author

chu11 commented Dec 21, 2018

rebased

@garlick
Copy link
Member

garlick commented Dec 21, 2018

Thanks!

@garlick garlick merged commit 56825e3 into flux-framework:master Dec 21, 2018
@chu11 chu11 deleted the issue1888 branch June 5, 2021 18:02
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.

None yet

3 participants