-
Notifications
You must be signed in to change notification settings - Fork 2
Feature/no param properties rule #4
base: master
Are you sure you want to change the base?
Conversation
indent rule does not yet allow setting a number of spaces. I made a note at the indent-rule reminding to use 4 spaces.
tslint-config-5minds/README.md
Outdated
3.3 **indent:** Enforces indentation spaces. | ||
|
||
3.3 **indent:** Enforces indentation spaces. As soon as this rule supports setting the number of spaces per indentation it will be set to 4 spaces in the 5Minds-tslint-config. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
da wir das noch diskutieren bitte diesen Absatz ändern - 4 spaces sind nur eine Möglichkeit
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
2 Space, da der meiste JS-Code bereits 2 beinhaltet und TypeScript sich daran anlehnen.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
bitte auf 2 Spaces anpassen und auch die tslint und .editorconfig für *.ts und *.js anpassen, danke
tslint-config-5minds/README.md
Outdated
class Foo { | ||
|
||
constructor(private bar: Bar) {} | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Würde hier noch eine Leerzeile einfügen, damit es konsistent zum obigen // good Beispiel ist.
Wie ist hier der Stand? Scheint ein wenig in Vergessenheit geraten zu sein :) |
Change requests wurden behandelt. Indent-size situation verbessert sich sobald wir tslint updaten und die richtige Regel nutzen (#12) (gehört aber nicht in diesen PR). |
Current version of our tslint includes new indent version which allows specifying number of spaces/tabs.
Verbietet folgenden Code: