Skip to content
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

Reference to MyProject.start without introducing it first on page 61 #17

Open
mguimas opened this issue Jan 4, 2018 · 2 comments
Open

Comments

@mguimas
Copy link

mguimas commented Jan 4, 2018

In page 61, when discussing the use of Mix, there is the sentence

"Starts the system; terminates as soon as MyProject.start finishes"

but "MyProject.start" was not introduced before.

Perhaps the introduction of Mix should be deferred until OTP applications are discussed to make the use of Mix more clear to the reader.

@sasa1977
Copy link
Owner

sasa1977 commented Jan 4, 2018

Thank you for the comment!

This section is just a brief mention of mix. The main reason I'm mentioning it here is to make people aware that there is a way of managing a project with multiple files. The mix tool isn't used in any other example before the chapter 7, so you need to read more than half of the book to see it in action. I was worried that people might get a wrong impression that we just shovel all the code into a single file, so I wanted to quickly mention it early on.

I do agree with you that explaining mix run is possibly redundant and confusing here. Since I'm just working on the update for the 2nd edition, I'll think about removing that.

Thanks again!

@mguimas
Copy link
Author

mguimas commented Jan 5, 2018

I don't think people will get the impression that Elixir programs are written in a single file, as no other serious language does that, and quickly, they will learn it from the text, so do not worry.

It is much preferable to introduce Mix in one shot later when it is really needed. It will be much helpful to recall Mix commands later after reading the book.

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

No branches or pull requests

2 participants