Update README per recommendations from Seth:

Update README per recommendations from Seth:
https://github.com/domokit/sky_sdk/pull/2
https://github.com/domokit/sky_sdk/pull/3

R=sethladd@google.com, ianh@google.com

Committed: https://chromium.googlesource.com/external/mojo/+/4a66a47cd460f6521e306fbd1ae513e67a931eee

[eseidel, 2015-04-15 18:34:38]

eseidel@chromium.org changed reviewers:
+ sethladd@google.com

[Hixie, 2015-04-15 19:10:31]

https://codereview.chromium.org/1091613002/diff/1/sky/sdk/README.md
File sky/sdk/README.md (right):

https://codereview.chromium.org/1091613002/diff/1/sky/sdk/README.md#newcode35
sky/sdk/README.md:35: Node build() => new Text('Hello, world!');
The reason not to do this is that in 99% of cases, this body will be bigger, and
so if we're expecting people to use this as a copypasta template, this is just
giving them more work (they'll have to put the curlies back).

https://codereview.chromium.org/1091613002/diff/1/sky/sdk/README.md#newcode65
sky/sdk/README.md:65: void main() async {
does this work? I sent feedback to dart-discuss asking for void async functions
but last I heard they had declined to support this.

[sethladd, 2015-04-15 19:20:19]

https://codereview.chromium.org/1091613002/diff/1/sky/sdk/README.md
File sky/sdk/README.md (right):

https://codereview.chromium.org/1091613002/diff/1/sky/sdk/README.md#newcode35
sky/sdk/README.md:35: Node build() => new Text('Hello, world!');
Totally fair.

https://codereview.chromium.org/1091613002/diff/1/sky/sdk/README.md#newcode65
sky/sdk/README.md:65: void main() async {
good catch. I should have removed the void annotation.

[eseidel, 2015-04-15 19:33:45]

So my understanding is that part #1 was done for copy/pasta support.

And Part #2 can't work due to a bug in dart?

So this patch should be discarded for now?

[sethladd, 2015-04-15 19:35:18]

Ignore #1 suggestion.

+1 for removing void as a return type annotation, and keeping async and await.

But low pri. If it's easy, please consider using async/await in these examples.

[eseidel, 2015-04-15 19:36:47]

Ian tells me removing void would be the same as:

Future<dynamic> main() async {

Will that still work?

[sethladd, 2015-04-15 19:46:52]

You can use Future or Future<dynamic>. IMO, since nothing listens to what main()
returns (generally speaking), you don't need to use a return type annotation at
all. I'm generally a fan of using type annotations, but I don't use a return
type annotation for main().

We're really discussing Sky's coding standards at this point. So definitely up
to you.

[eseidel, 2015-04-15 19:51:23]

ptal

[sethladd, 2015-04-15 19:57:11]

lgtm

thanks!

[eseidel, 2015-04-15 20:26:58]

Committed patchset #2 (id:20001) manually as
4a66a47cd460f6521e306fbd1ae513e67a931eee (presubmit successful).