Addition of examples on documentation for NewClient and NewClientFromPool

[filipecosta90]

• ensures that our users will have a quick way of knowing how to connect in a more advanced manner ( see Support for passing authentication options to redigo #67 )
• Added an example on how to use an SSL Connection with RediSearch
• Add the example from here ( and fixes it ): https://github.com/RediSearch/RediSearch/blob/master/docs/go_client.md
• Ensures we're testing with redisearch:edge instead of latest

[lgtm-com]

This pull request introduces 1 alert when merging 543495d into 51a0cb3 - view on LGTM.com

new alerts:

• 1 for Disabled TLS certificate check

[lgtm-com]

This pull request introduces 1 alert when merging 4624b96 into 51a0cb3 - view on LGTM.com

new alerts:

• 1 for Disabled TLS certificate check

[lgtm-com]

This pull request introduces 1 alert when merging a3b8936 into 51a0cb3 - view on LGTM.com

new alerts:

• 1 for Disabled TLS certificate check

[codecov]

Codecov Report

Merging #68 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #68   +/-   ##
=======================================
  Coverage   75.27%   75.27%           
=======================================
  Files          12       12           
  Lines        1197     1197           
=======================================
  Hits          901      901           
  Misses        233      233           
  Partials       63       63           

Impacted Files	Coverage Δ
redisearch/client.go	80.15% <ø> (ø)

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 69202df...163b264. Read the comment docs.

[lgtm-com]

This pull request introduces 1 alert when merging 7ea24ca into 51a0cb3 - view on LGTM.com

new alerts:

• 1 for Disabled TLS certificate check

[lgtm-com]

This pull request introduces 1 alert when merging 3dd6756 into 51a0cb3 - view on LGTM.com

new alerts:

• 1 for Disabled TLS certificate check

[filipecosta90]

This pull request introduces 1 alert when merging 3dd6756 into 51a0cb3 - view on LGTM.com

new alerts:

* 1 for Disabled TLS certificate check

@itamarhaber this alert is related to:

InsecureSkipVerify should not be used in production code.

that comes from the SSL connection example, due to InsecureSkipVerify, that is required for the SSL examples to work. Added the following comment:

       // InsecureSkipVerify controls whether a client verifies the
	// server's certificate chain and host name.
	// If InsecureSkipVerify is true, TLS accepts any certificate
	// presented by the server and any host name in that certificate.
	// In this mode, TLS is susceptible to man-in-the-middle attacks.
	// This should be used only for testing.

[lgtm-com]

This pull request introduces 1 alert when merging 248b636 into 69202df - view on LGTM.com

new alerts:

• 1 for Disabled TLS certificate check

[lgtm-com]

This pull request introduces 1 alert when merging 163b264 into 69202df - view on LGTM.com

new alerts:

• 1 for Disabled TLS certificate check

[itamarhaber]

Good that you're testing the example :_

[filipecosta90]

Good that you're testing the example :_

=)