{"action":"created","issue":{"url":"https://api.github.com/repos/MattiSG/Watai/issues/160","labels_url":"https://api.github.com/repos/MattiSG/Watai/issues/160/labels{/name}","comments_url":"https://api.github.com/repos/MattiSG/Watai/issues/160/comments","events_url":"https://api.github.com/repos/MattiSG/Watai/issues/160/events","html_url":"https://github.com/MattiSG/Watai/pull/160","id":118063263,"number":160,"title":"jscs@2.6.0 breaks build ⚠️","user":{"login":"greenkeeperio-bot","id":14790466,"avatar_url":"https://avatars.githubusercontent.com/u/14790466?v=3","gravatar_id":"","url":"https://api.github.com/users/greenkeeperio-bot","html_url":"https://github.com/greenkeeperio-bot","followers_url":"https://api.github.com/users/greenkeeperio-bot/followers","following_url":"https://api.github.com/users/greenkeeperio-bot/following{/other_user}","gists_url":"https://api.github.com/users/greenkeeperio-bot/gists{/gist_id}","starred_url":"https://api.github.com/users/greenkeeperio-bot/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/greenkeeperio-bot/subscriptions","organizations_url":"https://api.github.com/users/greenkeeperio-bot/orgs","repos_url":"https://api.github.com/users/greenkeeperio-bot/repos","events_url":"https://api.github.com/users/greenkeeperio-bot/events{/privacy}","received_events_url":"https://api.github.com/users/greenkeeperio-bot/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/MattiSG/Watai/labels/invalid","name":"invalid","color":"e6e6e6"}],"state":"closed","locked":false,"assignee":null,"milestone":null,"comments":1,"created_at":"2015-11-20T15:27:58Z","updated_at":"2015-11-23T23:25:18Z","closed_at":"2015-11-23T23:25:18Z","pull_request":{"url":"https://api.github.com/repos/MattiSG/Watai/pulls/160","html_url":"https://github.com/MattiSG/Watai/pull/160","diff_url":"https://github.com/MattiSG/Watai/pull/160.diff","patch_url":"https://github.com/MattiSG/Watai/pull/160.patch"},"body":"Hello :wave:\n\n:warning::warning::warning:\n\n[jscs](https://www.npmjs.com/package/jscs) just published its new version 2.6.0, which **is covered by your current version range**. After updating it in your project **the build kept failing**.\n\nThis means **it’s possible that your software is malfunctioning**, because of this update. Use this branch to work on adaptions and fixes.\n\n\nHappy fixing and merging :palm_tree:\n\n---\n[GitHub Release](https://github.com/jscs-dev/node-jscs/releases/tag/v2.6.0)\n\n<h2>Version <a href=\"http://urls.greenkeeper.io/jscs-dev/node-jscs/compare/v2.5.1...v2.6.0\">2.6.0</a> (11-18-2015):</h2>\n\n<p>Thanks to <a href=\"http://urls.greenkeeper.io/seanpdoyle\" class=\"user-mention\">@seanpdoyle</a>, we're able to move some of the ES6 rules from the <a href=\"http://urls.greenkeeper.io/dockyard/ember-suave\">ember-suave</a> preset to JSCS!</p>\n\n<h3>New Rules</h3>\n\n<h4>\n<a href=\"http://jscs.info/rule/disallowVar\"><code>disallowVar</code></a> (Sean Doyle)</h4>\n\n<p>Disallows declaring variables with <code>var</code>.</p>\n\n<p><code>\"disallowVar\": true</code></p>\n\n<div class=\"highlight highlight-source-js\"><pre><span class=\"pl-c\">// Valid</span>\n<span class=\"pl-k\">let</span> foo;\n<span class=\"pl-k\">const</span> <span class=\"pl-c1\">bar</span> <span class=\"pl-k\">=</span> <span class=\"pl-c1\">1</span>;</pre></div>\n\n<div class=\"highlight highlight-source-js\"><pre><span class=\"pl-c\">// Invalid</span>\n<span class=\"pl-k\">var</span> baz;</pre></div>\n\n<p>You can also use <code>\"disallowKeywords\": [\"var\"]</code></p>\n\n<h4>\n<a href=\"http://jscs.info/rule/requireArrayDestructuring\"><code>requireArrayDestructuring</code></a> (Sean Doyle)</h4>\n\n<p>Requires that variable assignment from array values are destructured.</p>\n\n<p><code>\"requireArrayDestructuring\": true</code></p>\n\n<div class=\"highlight highlight-source-js\"><pre><span class=\"pl-c\">// Valid</span>\n<span class=\"pl-k\">var</span> colors <span class=\"pl-k\">=</span> [<span class=\"pl-s\"><span class=\"pl-pds\">'</span>red<span class=\"pl-pds\">'</span></span>, <span class=\"pl-s\"><span class=\"pl-pds\">'</span>green<span class=\"pl-pds\">'</span></span>, <span class=\"pl-s\"><span class=\"pl-pds\">'</span>blue<span class=\"pl-pds\">'</span></span>];\n<span class=\"pl-k\">var</span> [ red ] <span class=\"pl-k\">=</span> colors;</pre></div>\n\n<div class=\"highlight highlight-source-js\"><pre><span class=\"pl-c\">// Invalid</span>\n<span class=\"pl-k\">var</span> colors <span class=\"pl-k\">=</span> [<span class=\"pl-s\"><span class=\"pl-pds\">'</span>red<span class=\"pl-pds\">'</span></span>, <span class=\"pl-s\"><span class=\"pl-pds\">'</span>green<span class=\"pl-pds\">'</span></span>, <span class=\"pl-s\"><span class=\"pl-pds\">'</span>blue<span class=\"pl-pds\">'</span></span>];\n<span class=\"pl-k\">var</span> red <span class=\"pl-k\">=</span> colors[<span class=\"pl-c1\">0</span>];</pre></div>\n\n<h4>\n<a href=\"http://jscs.info/rule/requireEnhancedObjectLiterals\"><code>requireEnhancedObjectLiterals</code></a> (Sean Doyle)</h4>\n\n<p>Requires declaring objects via ES6 enhanced object literals (shorthand versions of properties)</p>\n\n<p><code>\"requireEnhancedObjectLiterals\": true</code></p>\n\n<div class=\"highlight highlight-source-js\"><pre><span class=\"pl-k\">var</span> obj <span class=\"pl-k\">=</span> {\n  <span class=\"pl-en\">foo</span>() { },\n  bar\n};</pre></div>\n\n<div class=\"highlight highlight-source-js\"><pre><span class=\"pl-k\">var</span> obj <span class=\"pl-k\">=</span> {\n  <span class=\"pl-en\">foo</span><span class=\"pl-k\">:</span> <span class=\"pl-k\">function</span>() { },\n  bar<span class=\"pl-k\">:</span> bar\n};</pre></div>\n\n<h4>\n<a href=\"http://jscs.info/rule/requireObjectDestructuring\"><code>requireObjectDestructuring</code></a> (Sean Doyle)</h4>\n\n<p>Requires variable declarations from objects via destructuring</p>\n\n<p><code>\"requireObjectDestructuring\": true</code></p>\n\n<div class=\"highlight highlight-source-js\"><pre><span class=\"pl-c\">// Valid</span>\n<span class=\"pl-k\">var</span> { foo } <span class=\"pl-k\">=</span> SomeThing;\n<span class=\"pl-k\">var</span> { bar } <span class=\"pl-k\">=</span> SomeThing.foo;</pre></div>\n\n<div class=\"highlight highlight-source-js\"><pre><span class=\"pl-c\">// Invalid</span>\n<span class=\"pl-k\">var</span> foo <span class=\"pl-k\">=</span> SomeThing.foo;\n<span class=\"pl-k\">var</span> bar <span class=\"pl-k\">=</span> SomeThing.foo.bar;</pre></div>\n\n<h4>\n<a href=\"http://jscs.info/rule/disallowSpacesInGenerator\"><code>disallowSpacesInGenerator</code></a> (Francisc Romano)</h4>\n\n<p>Checks the spacing around the <code>*</code> in a generator function.</p>\n\n<div class=\"highlight highlight-source-js\"><pre><span class=\"pl-s\"><span class=\"pl-pds\">\"</span>disallowSpacesInGenerator<span class=\"pl-pds\">\"</span></span><span class=\"pl-k\">:</span> {\n    <span class=\"pl-s\"><span class=\"pl-pds\">\"</span>beforeStar<span class=\"pl-pds\">\"</span></span><span class=\"pl-k\">:</span> <span class=\"pl-c1\">true</span>,\n    <span class=\"pl-s\"><span class=\"pl-pds\">\"</span>afterStar<span class=\"pl-pds\">\"</span></span><span class=\"pl-k\">:</span> <span class=\"pl-c1\">true</span>\n}</pre></div>\n\n<div class=\"highlight highlight-source-js\"><pre><span class=\"pl-k\">var</span> <span class=\"pl-en\">x</span> <span class=\"pl-k\">=</span> <span class=\"pl-k\">function*</span>() {};\n<span class=\"pl-k\">function</span><span class=\"pl-k\">*</span><span class=\"pl-en\">a</span>() {};\n<span class=\"pl-k\">var</span> <span class=\"pl-en\">x</span> <span class=\"pl-k\">=</span> <span class=\"pl-k\">async</span> <span class=\"pl-k\">function*</span>() {};</pre></div>\n\n<h3>New Rule Options</h3>\n\n<ul>\n<li>\n<code>requireCamelCaseOrUpperCaseIdentifiers</code>: add <code>strict</code> option (Jan-Pieter Zoutewelle)</li>\n</ul>\n\n<p>Also forces the first character to not be capitalized.</p>\n\n<div class=\"highlight highlight-source-js\"><pre><span class=\"pl-s\"><span class=\"pl-pds\">\"</span>requireCamelCaseOrUpperCaseIdentifiers<span class=\"pl-pds\">\"</span></span><span class=\"pl-k\">:</span> {\n  <span class=\"pl-s\"><span class=\"pl-pds\">\"</span>strict<span class=\"pl-pds\">\"</span></span><span class=\"pl-k\">:</span> <span class=\"pl-c1\">true</span>\n}</pre></div>\n\n<div class=\"highlight highlight-source-js\"><pre><span class=\"pl-c\">// Valid</span>\n<span class=\"pl-k\">var</span> camelCase <span class=\"pl-k\">=</span> <span class=\"pl-c1\">0</span>;\n<span class=\"pl-k\">var</span> <span class=\"pl-c1\">UPPER_CASE</span> <span class=\"pl-k\">=</span> <span class=\"pl-c1\">4</span>;\n\n<span class=\"pl-c\">// Invalid</span>\n<span class=\"pl-k\">var</span> Mixed_case <span class=\"pl-k\">=</span> <span class=\"pl-c1\">2</span>;\n<span class=\"pl-k\">var</span> Snake_case <span class=\"pl-k\">=</span> { snake_case<span class=\"pl-k\">:</span> <span class=\"pl-c1\">6</span> };\n<span class=\"pl-k\">var</span> snake_case <span class=\"pl-k\">=</span> { SnakeCase<span class=\"pl-k\">:</span> <span class=\"pl-c1\">6</span> };</pre></div>\n\n<ul>\n<li>\n<code>disallowSpace(After|Before)Comma</code>: add <code>allExcept: ['sparseArrays']</code> (Brian Dixon)</li>\n<li>\n<code>validateQuoteMarks</code>: add \"ignoreJSX\" value (Oleg Gaidarenko)</li>\n<li>\n<code>requireMatchingFunctionName</code>: add <code>includeModuleExports</code> option (George Chung)</li>\n</ul>\n\n<h3>Fixes</h3>\n\n<ul>\n<li>\n<p>Account for sparse arrays in rules with spacing and commas (Brian Dixon)</p>\n\n<ul>\n<li><a href=\"http://jscs.info/rule/disallowCommaBeforeLineBreak\"><code>disallowCommaBeforeLineBreak</code></a></li>\n<li><a href=\"http://jscs.info/rule/requireCommaBeforeLineBreak\"><code>requireCommaBeforeLineBreak</code></a></li>\n</ul>\n</li>\n<li><p><code>requireSpaceBeforeBinaryOperators</code>: report \"operator =\" correctly when erroring (Rob Wu)</p></li>\n<li>\n<code>requireAlignedMultilineParams</code>: do not throw on function without body (Oleg Gaidarenko)</li>\n</ul>\n\n<h3>Preset Changes</h3>\n\n<ul>\n<li>\n<code>WordPress</code>: add <code>requireBlocksOnNewLines</code> (Gary Jones)</li>\n</ul>\n\n---\nThe new version differs by 23 commits .\n\n- [`99fa842`](https://github.com/jscs-dev/node-jscs/commit/99fa842447e0bcf019ac88a5d3192a4fe5707398) `2.6.0`\n- [`e36bd81`](https://github.com/jscs-dev/node-jscs/commit/e36bd81886ded2840627a71d8a767f9b0d17b88e) `Misc: add 2.6.0 changelog`\n- [`7a547b2`](https://github.com/jscs-dev/node-jscs/commit/7a547b250727e95c3654d7652204d276b1d31821) `requireSpacesInGenerator: port improved logic, fix spelling`\n- [`2b6a93c`](https://github.com/jscs-dev/node-jscs/commit/2b6a93cb0afed0b4bd401bb626f4aeca17f32681) `disallowSpacesInGenerator: improve logic and fix bug for shortland`\n- [`747c3b4`](https://github.com/jscs-dev/node-jscs/commit/747c3b4698aa538d3b776d0231d304d149d63783) `New Rule: disallowSpacesInGenerator`\n- [`f4bc106`](https://github.com/jscs-dev/node-jscs/commit/f4bc1063bd3d46d5cf37525214b699cf43c0647d) `requireAlignedMultilineParams: do not throw on function without body`\n- [`9c9d16a`](https://github.com/jscs-dev/node-jscs/commit/9c9d16a77178283e897af15a1264143aa7f6a4ea) `Docs: update publishing info`\n- [`91cc6d6`](https://github.com/jscs-dev/node-jscs/commit/91cc6d647e506f63103bb5bfc3560ea82b1d50e0) `Docs: improve a bit the docs for \"requireObjectKeysOnNewLine\" rule`\n- [`102399b`](https://github.com/jscs-dev/node-jscs/commit/102399b83706934a36922819182b417f4a6cb6df) `Misc: add new rules to the grouping.json`\n- [`3c2dbf7`](https://github.com/jscs-dev/node-jscs/commit/3c2dbf7eca2bf65b9985ff46d2c54317237c55d4) `New rule: `disallowVar``\n- [`ecec961`](https://github.com/jscs-dev/node-jscs/commit/ecec961417f2b1aef17fe10d2a2231c7f3d50717) `New rule: `requireArrayDestructuring``\n- [`b8969d4`](https://github.com/jscs-dev/node-jscs/commit/b8969d48584e169956422120d71be8f24e73f06b) `New rule: `requireEnhancedObjectLiterals``\n- [`bc6f960`](https://github.com/jscs-dev/node-jscs/commit/bc6f9608c78068c089e342b4b84c7fda98fa1552) `New rule: `requireObjectDestructuring``\n- [`ec8c0b0`](https://github.com/jscs-dev/node-jscs/commit/ec8c0b0eda7cc1aa427c1a6c77ded42adab39c8c) `requireMatchingFunctionName: add `includeModuleExports` option`\n- [`7e8c2af`](https://github.com/jscs-dev/node-jscs/commit/7e8c2afd2f4a3bdbffb690a358e4a05173372541) `requireSpaceBeforeBinaryOperators: report \"operator =\"`\n\n\nThere are 23 commits in total. See the [full diff](https://github.com/jscs-dev/node-jscs/compare/616d04b420676073e3835792543323ca5ba920c2...99fa842447e0bcf019ac88a5d3192a4fe5707398).\n\n---\nThis pull request was created by [greenkeeper.io](http://greenkeeper.io/).\nIt keeps your software, up to date, all the time.\n\n<sub>\nTired of seeing this sponsor message? Upgrade to the supporter plan!\nYou'll also get your pull requests faster :zap:\n</sub>"},"comment":{"url":"https://api.github.com/repos/MattiSG/Watai/issues/comments/159101906","html_url":"https://github.com/MattiSG/Watai/pull/160#issuecomment-159101906","issue_url":"https://api.github.com/repos/MattiSG/Watai/issues/160","id":159101906,"user":{"login":"MattiSG","id":222463,"avatar_url":"https://avatars.githubusercontent.com/u/222463?v=3","gravatar_id":"","url":"https://api.github.com/users/MattiSG","html_url":"https://github.com/MattiSG","followers_url":"https://api.github.com/users/MattiSG/followers","following_url":"https://api.github.com/users/MattiSG/following{/other_user}","gists_url":"https://api.github.com/users/MattiSG/gists{/gist_id}","starred_url":"https://api.github.com/users/MattiSG/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/MattiSG/subscriptions","organizations_url":"https://api.github.com/users/MattiSG/orgs","repos_url":"https://api.github.com/users/MattiSG/repos","events_url":"https://api.github.com/users/MattiSG/events{/privacy}","received_events_url":"https://api.github.com/users/MattiSG/received_events","type":"User","site_admin":false},"created_at":"2015-11-23T23:25:18Z","updated_at":"2015-11-23T23:25:18Z","body":"CI was not set up properly, invalid."}}