Repository navigation
Parens must be kept around sequential exprs with semicolons in record field assignments #17826
Description
Repro
let B = 2
// Analyzer suggests removing parens,
// but that changes the entire parsing of the expression…
let foo = {| A = ((); B = 3) |}
// …or causes a parsing error.
let bar = {| A = ((); ()) |}Expected behavior
SynExpr.shouldBeParenthesizedInContext should return true anywhere the semicolon is required for the sequential expression to compile:
One-liner sequential exprs
let B = 2
let foo = {| A = ((); B = 3) |}Indentation-sensitive/offsides sequential exprs
let B = 2
let foo =
{| A =
(();
B = 2) |}Actual behavior
It returns false, and the analyzer suggests removing unnecessary parentheses.
Activity
Fix shouldBeParenthesizedInContext for Sequential in record fields (#…
[automated] Merge branch 'main' => 'feature/net11-scouting' (#20039)
[feature/net11-scouting] Update dependencies from dotnet/arcade (#20034)