seegno / jscs-config-seegno

Seegno-flavored JSCS config
2 stars 0 forks source link

Disable `disallowKeywordsInComments` rule #21

Closed promag closed 8 years ago

promag commented 8 years ago

This change allows comments like // TODO and //FIXME.

madeiras commented 8 years ago

:-1: for general rule. Would rather use disallowKeywordsInComments: false for specific projects. /cc @ruimarinho @nunofgs @fixe

ruiquelhas commented 8 years ago

@promag do you still think this makes sense?

ruimarinho commented 8 years ago

Why is TODO and FIXME considered a keyword? Aren't those related to javascript keywords?

promag commented 8 years ago

@JoaoHenriquePereira it's common to have these comments when submitting WIP code. In those cases the commit can be created with no verifications I guess. Therefore I agree with your suggestion.