JS: Require curly braces in switch cases

versatica / mediasoup

Cutting Edge WebRTC Video Conferencing

https://mediasoup.org

ISC License

6.18k stars 1.12k forks source link

JS: Require curly braces in switch cases #1268

Closed ibc closed 9 months ago

ibc commented 9 months ago

Details

Require { } in each case block.

There is no real ESLint rule for it so we must use this:

'no-restricted-syntax'      : [ 2,
{
  'selector' : 'SwitchCase > *.consequent[type!="BlockStatement"]'
  'message'  : 'Switch cases without blocks are disallowed'
}
]

jmillan commented 9 months ago

@ibc, this rule is working fine: 'selector' : 'SwitchCase > *.consequent[type!="BlockStatement"]',

NOTE: It does not give those false positives.

ibc commented 9 months ago

'selector' : 'SwitchCase > *.consequent[type!="BlockStatement"]'

AMAZING, applied.