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

docs(toh-3/dart): copyedits and new snake_case glossary entry #1629

Merged
merged 2 commits into from
Jun 15, 2016

Conversation

chalin
Copy link
Contributor

@chalin chalin commented Jun 9, 2016

  • Copyedits made to glossary as well.
  • Note that "Scoped Package" has been removed from the Dart glossary.

Contributing to #1628.

@chalin chalin force-pushed the chalin-toh-3-dart-minor-edits-0608 branch 2 times, most recently from 683e761 to 30ca053 Compare June 9, 2016 20:42
@chalin chalin force-pushed the chalin-toh-3-dart-minor-edits-0608 branch from 30ca053 to b445fce Compare June 13, 2016 14:33
@wardbell
Copy link
Contributor

What is the state of this? It is difficult for me to review when there is a mix of Dart and TS changes.

The TS changes LGTM. Maybe it is better if Kathy merges this one.

@chalin
Copy link
Contributor Author

chalin commented Jun 13, 2016

Sorry, the TS-side change was for the glossary: I originally needed to add a new #docregion tag to use in the Dart-side glossary file. While in there I made some minor copy edits related to the "*-case" entries.

@kwalrath @thso : ready for your review.

@chalin chalin force-pushed the chalin-toh-3-dart-minor-edits-0608 branch from b445fce to 13cebac Compare June 13, 2016 20:57
chalin added 2 commits June 14, 2016 09:34
Copyedits made to glossary as well.
Note that "Scoped Package" has been removed from the Dart glossary.
@chalin chalin force-pushed the chalin-toh-3-dart-minor-edits-0608 branch from 13cebac to df54a19 Compare June 14, 2016 16:35
@kwalrath
Copy link
Contributor

Taking a look now...

@kwalrath
Copy link
Contributor

LGTM. Merging now.

@kwalrath kwalrath merged commit cd970c2 into angular:master Jun 15, 2016
@chalin chalin deleted the chalin-toh-3-dart-minor-edits-0608 branch June 15, 2016 20:29
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