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

Cleanup Mint README doc #140

Merged
merged 2 commits into from
Sep 19, 2017
Merged

Cleanup Mint README doc #140

merged 2 commits into from
Sep 19, 2017

Conversation

nitisht
Copy link
Contributor

@nitisht nitisht commented Sep 18, 2017

Fixes #135

@nitisht nitisht force-pushed the doc-cleanup branch 3 times, most recently from d19572c to ac23b7a Compare September 18, 2017 10:13
README.md Outdated
@@ -1,41 +1,16 @@
# Mint [![Slack](https://slack.minio.io/slack?type=svg)](https://slack.minio.io) [![Docker Pulls](https://img.shields.io/docker/pulls/minio/mint.svg?maxAge=604800)](https://hub.docker.com/r/minio/mint/)

Collection of tests to detect overall correctness of Minio server.
Mint detects overall correctness of Minio server, using Minio SDKs, minio client (mc), AWS SDKs and awscli. Use Mint to test Minio server instances for functional correctness.
Copy link
Contributor

Choose a reason for hiding this comment

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

comma between Minio server and using Minio SDKs can be removed.

README.md Outdated
|:--- |:--- |
|`name`| Name of SDK being tested |
|`function`| API signature being tested by the test case |
|`args`| Key value map of arguments being passed in the API being tests. Note that this is not exhaustive, and only significant arguments will be listed |
Copy link
Contributor

Choose a reason for hiding this comment

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

being passed in the API being tests ->redundant "being"

README.md Outdated
### Build using Travis

Each pull request when submitted to Github `travis-ci` runs build on mint to create new docker image on `play.minio.io`, our private docker registry. You can get the `mint` image associated with your pull request by just running `docker pull play.minio.io/mint:$PULL_REQUEST_SHA`. For example
**Note for Developers**: On each PR send to [Mint repository](https://github.com/minio/mint), `travis-ci` builds `mint` docker image and pushes it to `play.minio.io`, our private docker registry. You can get the `mint` image associated with your pull request by just running `docker pull play.minio.io/mint:$PULL_REQUEST_SHA`. For example
Copy link
Contributor

Choose a reason for hiding this comment

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

s/send/sent/

@nitisht
Copy link
Contributor Author

nitisht commented Sep 18, 2017

done @poornas

Copy link
Member

@harshavardhana harshavardhana left a comment

Choose a reason for hiding this comment

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

LGTM

@nitisht nitisht requested review from harshavardhana and removed request for deekoder September 19, 2017 02:15
@nitisht nitisht merged commit 800f085 into minio:master Sep 19, 2017
@nitisht nitisht deleted the doc-cleanup branch September 19, 2017 02:37
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.

3 participants