Skip to content

Updates pseudocode example usage from Mix.env to Mix.env() to avoid confusion #1533

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
Jun 18, 2021

Conversation

AlejandroFabianCampos
Copy link
Contributor

@AlejandroFabianCampos AlejandroFabianCampos commented Jun 18, 2021

I'm just getting into Elixir, so please bear with me if this is nonsense.

I was reading through the "introduction to Mix" guide, and https://github.com/elixir-lang/elixir-lang.github.com/blame/master/getting-started/mix-otp/introduction-to-mix.markdown#L283-L289 confused me slightly, since it is referencing that Mix.env is a function, but then not executing on the example usage.

By scrolling back, I could verify that the example mix.exs snippet https://github.com/elixir-lang/elixir-lang.github.com/blame/master/getting-started/mix-otp/introduction-to-mix.markdown#L85-L114, effectively points to executing Mix.env as function.

Sorry if this comes across like a little bit nitpicky, just trying to contribute back my two cents, love the docs!

…from Mix.env to Mix.env() to avoid confusion
@josevalim
Copy link
Member

This is really good and the code formatter would add those parens too. Thank you!

@josevalim josevalim merged commit 20da3aa into elixir-lang:master Jun 18, 2021
@josevalim
Copy link
Member

💚 💙 💜 💛 ❤️

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