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

Corrects README instructions regarding minimesos #944

Merged
merged 1 commit into from Aug 29, 2018

Conversation

Projects
None yet
2 participants
@dposada
Copy link
Member

dposada commented Aug 29, 2018

Changes proposed in this PR

  • switching from 1., 2., 3. to all 1. in the quickstart
  • recommending using the minimesos from /travis instead of installing your own

Why are we making these changes?

The numbering change is for easier editing of the list. The minimesos change is to encourage developers to use the bundled script instead of having to install their own.

@dposada dposada self-assigned this Aug 29, 2018

@dposada dposada removed the wip label Aug 29, 2018

@dposada dposada requested a review from shamsimam Aug 29, 2018

@shamsimam shamsimam merged commit af656f1 into twosigma:master Aug 29, 2018

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details

@dposada dposada deleted the dposada:readme-minimesos branch Aug 29, 2018

dposada added a commit to dposada/Cook that referenced this pull request Nov 1, 2018

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