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

Grammer mistake #271

Merged
merged 1 commit into from
Nov 30, 2016
Merged

Grammer mistake #271

merged 1 commit into from
Nov 30, 2016

Conversation

shade
Copy link

@shade shade commented Nov 28, 2016

Forgot an article.

@jwj61
Copy link
Member

jwj61 commented Nov 28, 2016

I think the original is ok. I am no grammer expert, and the phrasing seems to go well with the next phrase "square top of side ...", which also has no article.

@paultpearson
Copy link
Member

According to one online grammar checker I used, "pyramid with a square base of side b = ..." is preferable to "pyramid with square base of side b = ..."

I prefer "pyramid with a square base of side length b = ..."

@dlglin
Copy link
Member

dlglin commented Nov 28, 2016 via email

@shade
Copy link
Author

shade commented Nov 28, 2016

Perfect, someone should merge it then.

Also, @jwj61 in any case articles in lists are redundant and aren't used e.g. The code, comment, and argument of joe-tom were on point.

@paultpearson paultpearson merged commit a0a51b6 into openwebwork:master Nov 30, 2016
@shade shade deleted the patch-1 branch March 27, 2023 02:12
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.

4 participants