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

Fix typos, grammar and spelling in README.rst #6015

Merged
merged 1 commit into from
Nov 1, 2019

Conversation

ilyashamd
Copy link
Contributor

No description provided.

@CLAassistant
Copy link

CLAassistant commented Oct 31, 2019

CLA assistant check
All committers have signed the CLA.

@@ -112,7 +112,7 @@ You can run **conan** client and server in Windows, MacOS, and Linux.
search Searches package recipes and binaries in the local cache or in a remote.
Creator commands
new Creates a new package recipe template with a 'conanfile.py'.
create Builds a binary package for recipe (conanfile.py) located in current dir.
create Builds a binary package for a recipe (conanfile.py) located in the current dir.
Copy link
Member

Choose a reason for hiding this comment

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

These might be automatically obtained from the conan --help command, maybe it would make sense to fix them in the app code.

@@ -214,7 +214,7 @@ These should work for OS X:
$ export CONAN_COMPILER=clang
$ export CONAN_COMPILER_VERSION=3.5
Finally, there are some tests that use conan to package Go-lang
Finally, some tests use conan to package Go-lang
Copy link
Member

Choose a reason for hiding this comment

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

We have recently removed the go-lang tests, this section about go-lang might be removed.

@memsharded
Copy link
Member

This is good, many thanks!
Despite two minor comments, I am going to merge this, those things can be contributed later, and this is already an improvement.

@memsharded memsharded merged commit c8cc271 into conan-io:develop Nov 1, 2019
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