Skip to content

tools: enable space-after-keywords eslint rule #2430

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
brendanashworth wants to merge 2 commits into from
Closed

tools: enable space-after-keywords eslint rule #2430

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
f46676e
test: add spaces after keywords
brendanashworth Aug 18, 2015
2e2f255
tools: enable space-after-keywords in eslint
brendanashworth Aug 18, 2015
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Next Next commit
test: add spaces after keywords 
eg changes:

  if(x) { ... }

to:

  if (x) { ... }
  • Loading branch information
@brendanashworth
brendanashworth committed Aug 21, 2015
commit f46676efee6fc14b6445a5d847946594b7dc7915
2 changes: 1 addition & 1 deletion test/gc/test-http-client-timeout.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -54,7 +54,7 @@ function getall() {
setImmediate(getall);
}

for(var i = 0; i < 10; i++)
for (var i = 0; i < 10; i++)
getall();

function afterGC() {
Expand Down
2 changes: 1 addition & 1 deletion test/internet/test-dgram-broadcast-multi-process.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,7 @@ var common = require('../common'),
// take the first non-internal interface as the address for binding
get_bindAddress: for (var name in networkInterfaces) {
var interfaces = networkInterfaces[name];
for(var i = 0; i < interfaces.length; i++) {
for (var i = 0; i < interfaces.length; i++) {
var localInterface = interfaces[i];
if (!localInterface.internal && localInterface.family === 'IPv4') {
var bindAddress = localInterface.address;
Expand Down
2 changes: 1 addition & 1 deletion test/internet/test-dns.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -663,7 +663,7 @@ TEST(function test_resolve_failure(done) {
var req = dns.resolve4('nosuchhostimsure', function(err) {
assert(err instanceof Error);

switch(err.code) {
switch (err.code) {
case 'ENOTFOUND':
case 'ESERVFAIL':
break;
Expand Down
2 changes: 1 addition & 1 deletion test/parallel/test-net-listen-fd0.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@ process.on('exit', function() {

// this should fail with an async EINVAL error, not throw an exception
net.createServer(assert.fail).listen({fd:0}).on('error', function(e) {
switch(e.code) {
switch (e.code) {
case 'EINVAL':
case 'ENOTSOCK':
gotError = e;
Expand Down
2 changes: 1 addition & 1 deletion test/parallel/test-stream2-base64-single-char-read-end.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@ var accum = [];
var timeout;

src._read = function(n) {
if(!hasRead) {
if (!hasRead) {
hasRead = true;
process.nextTick(function() {
src.push(new Buffer('1'));
Expand Down
2 changes: 1 addition & 1 deletion test/parallel/test-util-inspect.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -106,7 +106,7 @@ assert.ok(util.inspect(y), '[ \'a\', \'b\', \'c\', \'\\\\\\\': \'d\' ]');
function test_color_style(style, input, implicit) {
var color_name = util.inspect.styles[style];
var color = ['', ''];
if(util.inspect.colors[color_name])
if (util.inspect.colors[color_name])
color = util.inspect.colors[color_name];

var without_color = util.inspect(input, false, 0, false);
Expand Down