Codelyzer: Whitespace rules in angular templates

[jaufgang]

the TSLint Whitespace rule can check for whitespace around operators, for example

if(foo===bar){return}  // TSLint missing whitespace error
const x=a?b:c;   //multiple TSLint missing whitespace errors

if(foo === bar){return} //no TSLint error
const x = a ? b : c; //no TSLint error

However, the codelyzer angular-whitespace rule does not catch this

<button *ngIf="foo===bar"
    [disabled]="checkCount()===0">
</button>  <!-- no TSLint error-->

Is there a way to catch this? Could the angular-whitespace rule be configured to piggyback off the whitespace rule, and enforce the same config options for code in angular templates as are used for the whitespace rule if it is configured?

[mgechev]

Following the feature request process that lodash has, I'm closing the issue and adding votes needed label.

Later we can prioritize the feature requests by popularity and include them in a future release.

[rafaelss95]

Some people, such as Material2 team, doesn't like whitespaces between the interpolations, so it could be another improvement: add some configuration like never, so that would validate it to have no spaces.

[jaufgang]

Another example of where whitespace rule could be enforced:

<div [ngClass]="{foo:item.isFoo}"> <!-- should have space after colon -->

[rafaelss95]

Thinking more about it... do we really want to messing with formatting rules in a linter? Well, while I can imagine how this rule could be useful in many cases, I really think it's more appropriated to left this kind of rule to a formatter, like Prettier*, for example.

IMHO, this project shouldn't mess with spaces and formatting in general. Formatting rules are always a pain and it also never satisfies every people.

* Sadly, Prettier still doesn't format HTML.

[rafaelss95]

Update: Now you can use Prettier to format HTML :)

@mgechev Let's remove the votes needed label here?

[mgechev]

@rafaelss95 you're a project collaborator. I believe you should have write access as well. Let me know if that's not the case so I can fix it.

[rafaelss95]

@mgechev Ah, yes I have :)