Minor doc changes in the Protocol.consolidate/2 #8518

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.

Sign up for GitHub

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

Jump to bottom

Merged

josevalim merged 1 commit into elixir-lang:master from fertapric:patch-10

Dec 13, 2018

Member

fertapric commented Dec 13, 2018

Changes:

The first element of the tuple is :ok if the protocol was consolidated.
Keep sentences that talk about return values together, and avoid possible misunderstandings with the return value of Protocol.consolidated?(Enumerable)


          Minor doc changes in the Protocol.consolidate/2

583a69a

[ci skip]

josevalim merged commit 31e177d into elixir-lang:master

Member

josevalim commented Dec 13, 2018

❤️ 💚 💙 💛 💜

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

Labels

None yet