Results 65 comments of Denis Bezrukov

Would you mind assigning me this task?πŸ™πŸ½

There is a conditional which is used to check `isLeftSideLowerPrecedence`. - https://github.com/typescript-eslint/typescript-eslint/blob/943523d9c345f01a7e285eb4440be2998614186d/packages/eslint-plugin/src/rules/prefer-optional-chain.ts#L73-L83 We need it for the case then it's necessary to add parentheses around left part of `LogicalExpression` after...

@ematipico Could you please explain how `FormatPrecedence::with_precedence_for_parenthesis` works now? Does It return `FormatPrecedence::High` for expressions which have more precedence over parentheses? And `FormatPrecedence::Low` for expressions which have less precedence over...

@ematipico Thank you for your explanation! I have concerns that there are cases then we can omit parenthesis for your first example but current implementation keeps them. E.g. ` a[(a...

@ematipico SorryπŸ˜… I hope that we can use the function which returns precedence in formatter and lint crate. Because it can handle cases which are inconsistent with prettier now. E.g....

Hi, I've handle cases with nullish coalescing and logical or: https://github.com/denbezrukov/tools/blob/feature/use-optional-chaining/crates/rome_js_analyze/tests/specs/js/useOptionalChain/nullishAndLogicalOr.ts.snap Now I'm working on: `foo && foo.bar && foo.bar.zoo`

Hi, I've handle cases with logical and. https://github.com/denbezrukov/tools/blob/feature/use-optional-chaining/crates/rome_js_analyze/tests/specs/js/useOptionalChain/logicalAndCases.js.snap I'm going to double check test cases, clean code and make MRπŸ₯³

- #3305 - https://github.com/rome/tools/pull/3474