New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

Fix minor errors found by Dialyzer #70

Merged

danielberkompas merged 2 commits into danielberkompas:master from soundmonster:dialyzer

Jun 14, 2019

Contributor

soundmonster commented Jun 4, 2019

Dialyzer found three call sites where atoms are used, but strings are expected:

the header map for HTTPoison.request/5 - its signature requires strings as keys
two functions in Elasticsearch.Index that expect strings for index name, but received atoms

Fixing this allows dependent projects to use Dialyzer without warnings.

soundmonster added 2 commits

June 3, 2019 18:07


          Adhere to HTTPoison.request/5 signature

6b30bcc


          Provide string version of alias name where expected

90c474d

soundmonster closed this

soundmonster reopened this

danielberkompas merged commit 1a7bbe7 into danielberkompas:master

# for free to join this conversation on GitHub. Already have an account? # to comment

Labels

None yet