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

Eliminate extra word #1442

Merged
merged 1 commit into from Nov 7, 2018

Conversation

Projects
None yet
2 participants
@dcrall
Contributor

dcrall commented Nov 6, 2018

"You can do achieve this" -> "You can achieve this"

Eliminate extra work
"You can do achieve this" -> "You can achieve this"
@dcrall

This comment has been minimized.

Contributor

dcrall commented on 5e20f1c Nov 6, 2018

Of course I made a typo on this pull request. Title should be "Eliminate extra word".

@ktsn

This comment has been minimized.

Member

ktsn commented Nov 7, 2018

Thanks for making the PR. However, do + verb looks valid English. It is used for emphasis.
https://www.usingenglish.com/forum/threads/166197-I-do-verb

@ktsn ktsn closed this Nov 7, 2018

@dcrall

This comment has been minimized.

Contributor

dcrall commented Nov 7, 2018

The example you sited is I do + verb, which is primarily a conversational construct. This edit does not match that pattern, and I can assure you it does not read as correct English. It is awkward, redundant, and adds no clarity to the statement

@dcrall

This comment has been minimized.

Contributor

dcrall commented Nov 7, 2018

Plus, in this sentence, you have three verbs: can, do, and achieve. That is another reason it doesn't match the pattern in the link.

@ktsn ktsn reopened this Nov 7, 2018

@ktsn ktsn merged commit 0fa5c9f into vuejs:dev Nov 7, 2018

6 checks passed

ci/circleci: install Your tests passed on CircleCI!
Details
ci/circleci: lint-types Your tests passed on CircleCI!
Details
ci/circleci: test-e2e Your tests passed on CircleCI!
Details
ci/circleci: test-ssr Your tests passed on CircleCI!
Details
ci/circleci: test-unit Your tests passed on CircleCI!
Details
deploy/netlify Deploy preview ready!
Details
@ktsn

This comment has been minimized.

Member

ktsn commented Nov 7, 2018

I see. Thank you for clarifying.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment