Skip to content

Minor documentation update - explicit mention of PID in GenServer.call/3 #14529

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

Merged
merged 1 commit into from
May 24, 2025

Conversation

silverdr
Copy link
Contributor

This PR comes as a result of discussion in elixirforum thread where the documentation for GenServer.call/3 was found to be lacking explicit mention of PID as a valid value for the server argument. Documentation sends reader to "Name registration" section, which in its vast majority covers the subject of naming and registering GenServers. While it in fact mentions the possibility of using PID in one of the paragraphs, it is arguably better to make a distinction as PID is neither a "name" nor something that the user can "register". This change should make it clear for situations where only PID is available, without deep referring to another section of the documentation, potentially overlooking the single mention of PID there.

@josevalim josevalim merged commit 9310ffc into elixir-lang:main May 24, 2025
@josevalim
Copy link
Member

Thank you! If there are other places which could benefit from the same change, another PR is welcome!

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

Successfully merging this pull request may close these issues.

2 participants