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

tools/rados: fixed typo in help information #15618

Merged
merged 1 commit into from Jun 17, 2017

Conversation

Projects
None yet
3 participants
@liupan1111
Contributor

liupan1111 commented Jun 12, 2017

Signed-off-by: Pan Liu wanjun.lp@alibaba-inc.com

@liupan1111

This comment has been minimized.

Contributor

liupan1111 commented Jun 12, 2017

@tchaikov Ping.

@liupan1111 liupan1111 added the cleanup label Jun 12, 2017

@joscollin

joscollin approved these changes Jun 12, 2017 edited

The changes Looks Good.

@joscollin joscollin added the tools label Jun 12, 2017

@@ -83,7 +83,7 @@ void usage(ostream& out)
"OBJECT COMMANDS\n"
" get <obj-name> [outfile] fetch object\n"
" put <obj-name> [infile] [--offset offset]\n"
" write object write object start offset(default:0)\n"

This comment has been minimized.

@tchaikov

tchaikov Jun 12, 2017

Contributor

better off adding a space before the opening parenthesis.

This comment has been minimized.

@liupan1111

liupan1111 Jun 12, 2017

Contributor

updated.

tools/rados: fixed redundant comment of help information
Signed-off-by: Pan Liu <wanjun.lp@alibaba-inc.com>

@joscollin joscollin changed the title from tools/rados: fixed redundant comment of help information to tools/rados: fixed typo in help information Jun 12, 2017

@tchaikov

This comment has been minimized.

Contributor

tchaikov commented Jun 16, 2017

@tchaikov

This comment has been minimized.

Contributor

tchaikov commented Jun 16, 2017

retest this please

2 similar comments
@tchaikov

This comment has been minimized.

Contributor

tchaikov commented Jun 16, 2017

retest this please

@liupan1111

This comment has been minimized.

Contributor

liupan1111 commented Jun 17, 2017

retest this please

@liupan1111

This comment has been minimized.

Contributor

liupan1111 commented Jun 17, 2017

@tchaikov Jenkins passed.

@tchaikov tchaikov merged commit 53c04e5 into ceph:master Jun 17, 2017

4 checks passed

Signed-off-by all commits in this PR are signed
Details
Unmodified Submodules submodules for project are unmodified
Details
arm64 make check arm64 make check succeeded
Details
make check make check succeeded
Details

@liupan1111 liupan1111 deleted the liupan1111:wip-fix-comment branch Jun 17, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment