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

Fix typos in preface #16

Merged
merged 11 commits into from
Feb 15, 2017
Merged

Fix typos in preface #16

merged 11 commits into from
Feb 15, 2017

Conversation

ryangalamb
Copy link
Contributor

Thanks for writing this book! I'm really enjoying it so far.

I'm fixing any typos and grammatical errors I catch as I work my way through the book. These are my fixes for the preface.

I kept each fix in a separate commit to make it easier to review, but I can squash them all together if you'd prefer to keep the commit history less noisy.

@ryangalamb
Copy link
Contributor Author

And please let me know if there are any places where you disagree with how I "fixed" something. I tried to keep my changes as close to your words as possible, but there were some places where I might have deviated from your intentions.

@@ -502,7 +502,7 @@ There are many books on this topic and even courses out there already, made
book, and probably the students are assumed to obtain enough hardware knowledge
beforehand from other courses.
Or, the students only need to work within existing assignment framework:
the only thing they need to do is filling the empty function body, run
the only thing they need to do is fill the empty function body, run
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Isn't fill a verb already?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I wish I could give you a more technical answer, but I'm honestly not sure of the exact reasons why "fill" is correct here. It's correct for the same reasons "run" and "complete" are correct in the same sentence though.

@tuhdo
Copy link
Owner

tuhdo commented Feb 15, 2017

Thank you very much!

@tuhdo tuhdo merged commit 87a804a into tuhdo:master Feb 15, 2017
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

Successfully merging this pull request may close these issues.

2 participants