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

Update ch1.md #1431

Open
wants to merge 1 commit into
base: master
from

Conversation

Projects
None yet
2 participants
@deepakcdo
Copy link

deepakcdo commented Mar 3, 2019

Made it clear that a was requested while executing function foo

Yes, I promise I've read the Contributions Guidelines (please feel free to remove this line).

Update ch1.md
Made it clear that `a` was requested while executing function foo
@@ -159,7 +159,7 @@ Let's imagine the above exchange (which processes this code snippet) as a conver
> ***Engine***: Great, thanks! OK, I'm executing `foo`.
> ***Engine***: Hey, *Scope*, I've got an LHS reference for `a`, ever heard of it?
> ***Engine***: Hey, *Scope*, While executing function `foo` I've got an LHS reference for `a`, ever heard of it?

This comment has been minimized.

@getify

getify Mar 3, 2019

Owner

Is the "I'm executing foo." in the line immediately above it not clear enough?

This comment has been minimized.

@deepakcdo

deepakcdo Mar 3, 2019

Author

When I first read it I was not full sure what stage of exectuing function foo was at. When I read it the scond time I understood it. So I thought it would be clear to add that happens while execution the function.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.