Skip to content

Update riak admin usage and examples to reflect changes #1069

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 2 commits into from
Jul 15, 2021
Merged

Update riak admin usage and examples to reflect changes #1069

merged 2 commits into from
Jul 15, 2021

Conversation

@JMercerGit
Copy link
Contributor

Since the format of riak-admin was changed to riak admin, examples such as the usage prompt from running riak admin need to be corrected.
This closes issue #1068

@JMercerGit
Update riak admin usage and examples to reflect changes
5123c53 
Since the format of `riak-admin` was changed to `riak admin`, examples such as the usage prompt from running `riak admin` need to be corrected.
Bob-The-Marauder
Bob-The-Marauder reviewed May 19, 2021
View reviewed changes
Copy link
Contributor

@Bob-The-Marauder Bob-The-Marauder left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think you may have been a little over-zealous here. e.g. SCRIPT="riak-admin" should be correct as is. The rest look good to me though

@JMercerGit
Copy link
Contributor Author

I think you may have been a little over-zealous here. e.g. SCRIPT="riak-admin" should be correct as is. The rest look good to me though

Thanks, My mistake. I've corrected above.

Bob-The-Marauder
Bob-The-Marauder reviewed Jul 9, 2021
View reviewed changes
Copy link
Contributor

@Bob-The-Marauder Bob-The-Marauder left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me now

@martinsumner martinsumner merged commit 7d78666 into basho:develop-3.0 Jul 15, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer

@Bob-The-Marauder Bob-The-Marauder Bob-The-Marauder left review comments

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@JMercerGit @Bob-The-Marauder @martinsumner