Victorien Elvinger

feat(rome_js_analyze): `noUselessSwitchCase`

1

## Summary This implements *Unicorn*'s [no-useless-switch-case](https://github.com/sindresorhus/eslint-plugin-unicorn/blob/main/docs/rules/no-useless-switch-case.md) with the difference that we also handle cases after `default`. This makes the *Rome*'s rule more complete than *Unicorn*'s one. ## Impl choices I...

### Description [no-sequences](https://eslint.org/docs/latest/rules/no-sequences) ## Suggested name `noComaSequences`

### Description This should be a *good-first-issue*. [no-confusing-arrow](https://eslint.org/docs/latest/rules/no-confusing-arrow)

### Description This should be a *good-first-issue*. [no-unsafe-negation](https://eslint.org/docs/latest/rules/no-unsafe-negation) ## Suggested name `noUnsafeLogicNegation`

feat(rome_js_analyze): `noAssignInExpressions`

7

## Summary This is an exclusive rule for *Rome*! This disallows any assignment in an expression. This covers two ESLint's rules: [no-cond-assign](https://eslint.org/docs/latest/rules/no-cond-assign) and [no-return-assign](https://eslint.org/docs/latest/rules/no-return-assign). ## Implemenattion alternatives The rule could...

### Description This should be a *good-first-issue*. [no-this-before-super](https://eslint.org/docs/latest/rules/no-this-before-super).

### Environment information ```block Rome latest. ``` ### What happened? Rome amits an error for the following snippet of code: ```ts type B = A extends infer T extends B...

feat(rome_js_analyze): `noSequences`

2

## Summary Closes: #3932 This implements ESlint's rule [no-sequences](https://eslint.org/docs/latest/rules/no-sequences). In contrast to *ESLint*, we do not support the option `allowInParentheses` (enabled by default in *ESLint* ; disabled in *Rome*). I...

🐛 missing syntax highlighting on the playground

1

### Environment information ```block Playground ``` ### What happened? The syntax highlighting in the [playground](https://docs.rome.tools/playground/?quoteStyle=single&code=ZQB4AHAAbwByAHQAIABjAG8AbgBzAHQAIABYACAAPQAgADUA) seems no longer working... ### Expected result Syntax highlighting should work. ### Code of Conduct...

📎 Implement `noThisAlias`, @typescript-eslint/no-this-alias

7

### Description Implement the recommended ESLint rule [@typescript-eslint/no-this-alias](https://typescript-eslint.io/rules/no-this-alias/). Note that, we should accept restructuring assignment such as `const { props, state } = this`.