seegno / jscs-config-seegno

Seegno-flavored JSCS config
2 stars 0 forks source link

Update `requirePaddingNewlinesBeforeKeywords` rule #7

Closed ruiquelhas closed 8 years ago

ruiquelhas commented 8 years ago

Resolves #4.

nunofgs commented 8 years ago

I think we should try to get this fixed in JSCS rather than disable it. Otherwise, if we merge this, then this will be considered valid:

function one() {
}
function two() {
}

When, in fact, we want a newline before each function.

ruiquelhas commented 8 years ago

Actually, in this specific case, I guess because we have requirePaddingNewLinesAfterBlocks, that code snippet will be invalid.

ruipenso commented 8 years ago

@ruiquelhas Please rebase.