Closure compilation: rewrite doc now that there's try bots

Closure compilation: rewrite doc now that there's try bots

* De-emphasize local instructions
* Remove some instructions that are probably bitrotten at this point :(

R=groby@chromium.org
BUG=468366
NOTRY=true

Committed: https://crrev.com/b0b126710575e302541eb44c7a5d0c4c1000ffa5
Cr-Commit-Position: refs/heads/master@{#391362}

[Dan Beam, 2016-05-03 04:14:45]

Description was changed from

==========
Closure compilation: rewrite doc now that there's try bots

De-emphasize local instructions
Remove some instructions that are probably bitrotten at this point :(

R=groby@chromium.org
BUG=468366
==========

to

==========
Closure compilation: rewrite doc now that there's try bots

* De-emphasize local instructions
* Remove some instructions that are probably bitrotten at this point :(

R=groby@chromium.org
BUG=468366
==========

[groby-ooo-7-16, 2016-05-03 05:51:15]

Thank you for updating this.

LGTM with a few nitpicky nits.

https://codereview.chromium.org/1942133002/diff/1/docs/closure_compilation.md
File docs/closure_compilation.md (right):

https://codereview.chromium.org/1942133002/diff/1/docs/closure_compilation.md...
docs/closure_compilation.md:5: [Strongly-typed
languages](https://en.wikipedia.org/wiki/Strong_and_weak_typing)
Completely unnecessary nitpick of the week: Strictly speaking, c++ is static,
not strong. (Since you can cast all you want)

https://codereview.chromium.org/1942133002/diff/1/docs/closure_compilation.md...
docs/closure_compilation.md:21: When differently-typed variables interact with
eachother, the compiler can warn
nit:each[space]other

https://codereview.chromium.org/1942133002/diff/1/docs/closure_compilation.md...
docs/closure_compilation.md:30: JavaScript is loosely-typed and doesn't offer
this safety by default. This makes
weakly-typed? Since that's what you start of with?

https://codereview.chromium.org/1942133002/diff/1/docs/closure_compilation.md...
docs/closure_compilation.md:174: this file is used by the
[T]his...

https://codereview.chromium.org/1942133002/diff/1/docs/closure_compilation.md...
docs/closure_compilation.md:176: will automatically compile your code on every
commit.
And will automatically... or It will automatically...

https://codereview.chromium.org/1942133002/diff/1/docs/closure_compilation.md...
docs/closure_compilation.md:184: From the command line, you can issue a try job
with your current diff by:
s/diff/changes

https://codereview.chromium.org/1942133002/diff/1/docs/closure_compilation.md...
docs/closure_compilation.md:197: Working in common resource directories in
Chrome automatically add this line for
add[s]

[Dan Beam, 2016-05-03 21:24:06]

https://codereview.chromium.org/1942133002/diff/1/docs/closure_compilation.md
File docs/closure_compilation.md (right):

https://codereview.chromium.org/1942133002/diff/1/docs/closure_compilation.md...
docs/closure_compilation.md:5: [Strongly-typed
languages](https://en.wikipedia.org/wiki/Strong_and_weak_typing)
On 2016/05/03 05:51:15, groby wrote:
> Completely unnecessary nitpick of the week: Strictly speaking, c++ is static,
> not strong. (Since you can cast all you want)
> 

Acknowledged.

https://codereview.chromium.org/1942133002/diff/1/docs/closure_compilation.md...
docs/closure_compilation.md:21: When differently-typed variables interact with
eachother, the compiler can warn
On 2016/05/03 05:51:15, groby wrote:
> nit:each[space]other

Done.

https://codereview.chromium.org/1942133002/diff/1/docs/closure_compilation.md...
docs/closure_compilation.md:30: JavaScript is loosely-typed and doesn't offer
this safety by default. This makes
On 2016/05/03 05:51:15, groby wrote:
> weakly-typed? Since that's what you start of with?

Done.

https://codereview.chromium.org/1942133002/diff/1/docs/closure_compilation.md...
docs/closure_compilation.md:174: this file is used by the
On 2016/05/03 05:51:15, groby wrote:
> [T]his...

Done.

https://codereview.chromium.org/1942133002/diff/1/docs/closure_compilation.md...
docs/closure_compilation.md:176: will automatically compile your code on every
commit.
On 2016/05/03 05:51:15, groby wrote:
> And will automatically... or It will automatically...

Done. (changed to "used by the closure compiler bot to automatically")

https://codereview.chromium.org/1942133002/diff/1/docs/closure_compilation.md...
docs/closure_compilation.md:184: From the command line, you can issue a try job
with your current diff by:
On 2016/05/03 05:51:15, groby wrote:
> s/diff/changes

Done. (From the command line, you try your change with:)

https://codereview.chromium.org/1942133002/diff/1/docs/closure_compilation.md...
docs/closure_compilation.md:197: Working in common resource directories in
Chrome automatically add this line for
On 2016/05/03 05:51:15, groby wrote:
> add[s]

Done.

[Dan Beam, 2016-05-03 21:27:30]

Description was changed from

==========
Closure compilation: rewrite doc now that there's try bots

* De-emphasize local instructions
* Remove some instructions that are probably bitrotten at this point :(

R=groby@chromium.org
BUG=468366
==========

to

==========
Closure compilation: rewrite doc now that there's try bots

* De-emphasize local instructions
* Remove some instructions that are probably bitrotten at this point :(

R=groby@chromium.org
BUG=468366
NOTRY=true
==========

[Dan Beam, 2016-05-03 21:27:34]

The CQ bit was checked by dbeam@chromium.org

[Dan Beam, 2016-05-03 21:27:35]

The patchset sent to the CQ was uploaded after l-g-t-m from groby@chromium.org
Link to the patchset: https://codereview.chromium.org/1942133002/#ps40001
(title: "change order of try and continous integration systems")

[commit-bot: I haz the power, 2016-05-03 21:27:52]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1942133002/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1942133002/40001

[commit-bot: I haz the power, 2016-05-03 21:33:30]

Description was changed from

==========
Closure compilation: rewrite doc now that there's try bots

* De-emphasize local instructions
* Remove some instructions that are probably bitrotten at this point :(

R=groby@chromium.org
BUG=468366
NOTRY=true
==========

to

==========
Closure compilation: rewrite doc now that there's try bots

* De-emphasize local instructions
* Remove some instructions that are probably bitrotten at this point :(

R=groby@chromium.org
BUG=468366
NOTRY=true
==========

[commit-bot: I haz the power, 2016-05-03 21:33:31]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-05-03 21:35:34]

Description was changed from

==========
Closure compilation: rewrite doc now that there's try bots

* De-emphasize local instructions
* Remove some instructions that are probably bitrotten at this point :(

R=groby@chromium.org
BUG=468366
NOTRY=true
==========

to

==========
Closure compilation: rewrite doc now that there's try bots

* De-emphasize local instructions
* Remove some instructions that are probably bitrotten at this point :(

R=groby@chromium.org
BUG=468366
NOTRY=true

Committed: https://crrev.com/b0b126710575e302541eb44c7a5d0c4c1000ffa5
Cr-Commit-Position: refs/heads/master@{#391362}
==========

[commit-bot: I haz the power, 2016-05-03 21:35:35]

Patchset 3 (id:??) landed as
https://crrev.com/b0b126710575e302541eb44c7a5d0c4c1000ffa5
Cr-Commit-Position: refs/heads/master@{#391362}