Update README.md #668
Update README.md #668
Conversation
|
It looks like @andreyferriyan hasn't signed our Contributor License Agreement, yet.
You can read and sign our full Contributor License Agreement here. Once you've signed reply with Appreciation of efforts, clabot |
|
|
@confluentinc It looks like @andreyferriyan just signed our Contributor License Agreement. 👍 Always at your service, clabot |
README.md
Outdated
|
|
||
| To avoid error in installation like *no matches found: confluent-kafka[avro]*, use double quotes. As an example: | ||
|
|
||
| $ pip install "confluent-kafka[avro]" |
There was a problem hiding this comment.
I think it is better to just change the "pip install ..avro]" line above to include quotes, people tend to cutnpaste the first command they see
There was a problem hiding this comment.
Sorry, this is my first time to change. Where should I change the line ?. In my forking repo ?
There was a problem hiding this comment.
Yes, undo your previous changes and modify the original pip install confluent-kafka[avro] line above to have quotes. Commit and push.
README.md
Outdated
|
|
||
| **Install from source from PyPi** *(requires librdkafka + dependencies to be installed separately)*: | ||
|
|
||
| $ pip install --no-binary :all: confluent-kafka | ||
|
|
||
|
|
||
| For source install, see *Prerequisites* below. | ||
| For source install, see *Prerequisites* below. |
There was a problem hiding this comment.
remove the whitespace diff
There was a problem hiding this comment.
This whitespace change needs to be removed too.
Update README.md in installation section. To avoid error with different shell.