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

eek -> eke #1689

Open
wants to merge 3 commits into
base: 2nd-ed
Choose a base branch
from
Open

eek -> eke #1689

wants to merge 3 commits into from

Conversation

philip-healy
Copy link

This is a typo fix. I have read the contributing guidelines that advise against typo fixes (specifically: "these will almost certainly be caught during the editing process"). However, I feel justified in submitting a PR for this one because it made it through the editing process to my printed copy of the second edition! Great book BTW. If I come across any other typos I will add them as additional commits to this PR.

Typos?

Please don't worry about minor text typos. These will almost certainly be caught during the editing process.

If you're going to submit a PR for typo fixes, please be measured in doing so by collecting several small changes into a single PR (in separate commits). Or, just don't even worry about them for now, because we'll get to them later. I promise.


Please type "I already searched for this issue":
I already searched for this issue

Edition: (pull requests not accepted for previous editions)
2nd

Book Title:
Scope & Closures

Chapter:
7

Section Title:
Per Variable or Per Scope?

Topic:
Typo fix!

@getify
Copy link
Owner

getify commented Aug 24, 2020

Good catch, thanks.

@getify getify added the errata label Aug 24, 2020
@getify getify force-pushed the 2nd-ed branch 9 times, most recently from 7916ced to 57e740a Compare January 25, 2022 23:53
@getify getify force-pushed the 2nd-ed branch 12 times, most recently from bf58419 to bae21ee Compare February 17, 2022 00:02
@getify getify force-pushed the 2nd-ed branch 4 times, most recently from 01b2cc1 to 25c4059 Compare July 16, 2022 14:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants