Publish new release with support for updated style guide

[addyosmani]

• Add support for new style guide to master
• Test master with Google OSS projects (e.g sw-toolbox) that use the style guide to determine state of breakage, if any. What else might be useful to capture in the changelog notes
• Tag new major release and publish to npm

cc @philipwalton @gauntface as an fyi

[philipwalton]

I updated a few projects I work on with the new style guide and found three major areas of breakage:

1. comma-dangle: New styles require trailing commas in arrays and object definition blocks. Many of our OSS projects do not do this.

2. arrow-parens: The styleguide recommends using parens around single-argument arrow functions. Several of the projects I've seen extending this config include rules that actually require no parens in those cases. This may not cause breakage (if the rule is an override), but we should be consistent.

3. no-invalid-this: I don't think this is a change for the previous Google style guide, closure compiler has always been pretty strict about where you could use this for optimization reasons.

[gauntface]

Is say make it a major release and don't worry about the breakage. If it's best practice / style then go with it.

[BrockWills]

Is there any way to view the new styleguide? The style guide linked in the readme is no longer accurate with these changes

[addyosmani]

It hasn't been published outside of Google just yet. We're working on it but it's going to take a bit :)

[appsforartists]

CC @dongbohu

[gauntface]

Worth updating the readme to hint at using the eslint:recommended config as well as Google?

Found a few things that were nice to have from eslint:recommnded - forced env + ecmascript config.

[addyosmani]

I think so. Could you file an issue and we can get that tackled?
On Tue, Oct 18, 2016 at 7:16 PM Matt Gaunt [email protected] wrote:

Worth updating the readme to hint at using the eslint:recommended config
as well as Google?

Found a few things that were nice to have from eslint:recommnded - forced
env + ecmascript config.

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
#24 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAGxaUijiOL73qPTpYT8wrwqSSRMLBXmks5q1X1lgaJpZM4KHHen
.

[philipwalton]

Created #27. I'll send a PR for this shortly.

[ilyaigpetrov]

0.7.1 was published on NPM.
Please, update url of "Google JavaScript style guide" in README.md
@addyosmani and all the team, thank you.

[goatandsheep]

New style guide published publicly!