Skip to content
This repository was archived by the owner on Dec 4, 2017. It is now read-only.

Kwong glossary updates #2292

Closed

Conversation

kapunahelewong
Copy link
Contributor

Edits to Glossary in docs/ts/latest/glossary.jade.

By compiling your application using the compiler-cli, `ngc`, you can bootstrap directly
to a Module Factory, meaning you don't need to include the Angular compiler in your javascript bundle.
to a Module Factory, meaning you don't need to include the Angular compiler in your Javascript bundle.
Copy link
Contributor

Choose a reason for hiding this comment

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

Global: Javascript -> JavaScript

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done. This was the only one that got through. :)

@kwalrath
Copy link
Contributor

kwalrath commented Sep 8, 2016

The changes you've made look good, but there are many more changes you could make—especially ones that came up in reviews of previous pages you worked on. I marked a few things, but I wasn't too thorough this time.

If you could refer to the checklist (and, better yet, improve the checklist) each time you edit a doc, I think that'd improve the quality of the docs.

@@ -28,26 +28,25 @@ block includes
+ifDocsFor('ts')
a#aot
:marked
## Ahead of Time (AOT) Compilation
## Ahead of Time (AOT) compilation
Copy link
Contributor

Choose a reason for hiding this comment

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

Time -> time

Copy link
Contributor

Choose a reason for hiding this comment

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

In fact, let's make this "Ahead-of-time", since it's a adjectival phrase before a noun. (http://www.grammarbook.com/punctuation/hyphens.asp)

Copy link
Contributor

Choose a reason for hiding this comment

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

should actually by AoT

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done. (And bookmarked that page - thanks!)

@kwalrath
Copy link
Contributor

Looks great! I found a few more capitalization/hyphenation things, but once you fix those, LGTM.

@kapunahelewong kapunahelewong force-pushed the kwong-glossary-updates branch 3 times, most recently from d487821 to eb9eb10 Compare September 14, 2016 15:52
@kwalrath
Copy link
Contributor

LGTM once you resolve the conflicts.

@kapunahelewong kapunahelewong force-pushed the kwong-glossary-updates branch 5 times, most recently from b5b44d6 to cc886f6 Compare September 14, 2016 18:57
@kapunahelewong
Copy link
Contributor Author

Having issues resolving conflicts. Testing starting over in a new PR. I have copied the contents of glossary.jade as they are in this repo currently and am pushing up in #2346

@kwalrath kwalrath closed this in 4d46d7c Sep 14, 2016
@kwalrath kwalrath deleted the kwong-glossary-updates branch September 14, 2016 20:38
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