Skip to content
This repository was archived by the owner on Apr 12, 2024. It is now read-only.

docs(guide/Unit Testing): tighter English #10478

Closed
wants to merge 1 commit into from
Closed

docs(guide/Unit Testing): tighter English #10478

wants to merge 1 commit into from

Conversation

mcknell
Copy link
Contributor

@mcknell mcknell commented Dec 16, 2014

Leaner, more forceful style.

Fixes a grammatical problems with subject number doesn't agrees with the verb. (originally "We have built many features into
Angular which makes...", which reduces to "features...which makes").

More authoritative use of commas, such as the Oxford comma in lists of three or more.

Sentences that are sentences, not fragments. (Yes, that was a fragment just now. I'm not writing docs now that reflect the polish of the project.)

The English grammatical/stylistic notion of parallelism.

Etc. This is just polish. The ideas are fine.

Leaner, more forceful style. 

Fixes a grammatical problems with subject number doesn't agrees with the verb. (originally "We have built many features into
Angular which makes...", which reduces to "features...which makes"). 

More authoritative use of commas, such as the Oxford comma in lists of three or more. 

Sentences that are sentences, not fragments. (Yes, that was a fragment just now. I'm not writing docs now that reflect the polish of the project.)

The English grammatical/stylistic notion of parallelism. 

Etc. This is just polish. The ideas are fine.
@googlebot
Copy link

Thanks for your pull request.

It looks like this may be your first contribution to a Google open source project, in which case you'll need to sign a Contributor License Agreement (CLA) at https://cla.developers.google.com/.

If you've already signed a CLA, it's possible we don't have your GitHub username or you're using a different email address. Check the information on your CLA or see this help article on setting the email on your git commits.

Once you've done that, please reply here to let us know. If you signed the CLA as a corporation, please let us know the company's name.

@pkozlowski-opensource
Copy link
Member

@mcknell could you please sign the CLA so we can merge the proposed changes? Thnx!

@pkozlowski-opensource pkozlowski-opensource added this to the Purgatory milestone Dec 18, 2014
@pkozlowski-opensource
Copy link
Member

@mcknell did you sign the CLA?

@caitp caitp modified the milestones: Backlog, Purgatory Jan 6, 2015
@caitp
Copy link
Contributor

caitp commented Jan 6, 2015

still waiting on a CLA signature @mcknell :'(.

Just a note though, this seems to introduce some very long run-on sentences, it would probably be good to rewrite those better.

@Narretz Narretz self-assigned this Jun 11, 2015
@Narretz
Copy link
Contributor

Narretz commented Sep 18, 2015

@mcknell Since you've never signed the CLA, would you be okay with me committing these changes with my identity?

@mcknell
Copy link
Contributor Author

mcknell commented Sep 18, 2015

When I go to view my CLA at https://cla.developers.google.com/clas , it indicates I signed a "Google Individual CLA" on May 17, 2015. I just added an "Android Individual CLA" today -- not sure why that would help, but thought I'd frob the system in case your searches were somehow stuck.
But sure, take the changes in any form you like.

 On Friday, September 18, 2015 11:41 AM, Martin Staffa <[email protected]> wrote:

@mcknell Since you've never signed the CLA, would you be okay with me committing these changes with my identity?—
Reply to this email directly or view it on GitHub.

@googlebot
Copy link

CLAs look good, thanks!

provide dependency injection for your XHR requests, which can be mocked, and we provide abstractions which
allow you to test your model without having to resort to manipulating the DOM. The test can then
assert that the data has been sorted without having to create or look at the state of the DOM or
With Angular, we try to make it easy for you to do the right thing. For your XHR requests, we
Copy link
Contributor

Choose a reason for hiding this comment

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

XHR requests is redundant, speaking of tighter english =) But I think it's okay anyways.

@caitp caitp closed this in ef3df93 Sep 18, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants