8$: Extra character `,` on end of useState() gives mediocre error message.

[kyle-wannacott]

Doesn't tell me to remove extra character , on the end.

This should give an error saying unexpected token and missing semicolon like below:

[strager]

I think we can implement this using the following logic: if const is after , when parsing variable bindings (parser::parse_and_visit_let_bindings in parse-statement-inl.h), reporting a different error than E0024 (error_let_with_no_bindings). I think E0036 is similar.

[ghost]

I would like to give this issue a shot.

I claim this for-hire task. I expect payment after I complete this task. I will email the quick-lint-js team if I am assigned this task.

[Moshahrani]

I think we can implement this using the following logic: if const is after , when parsing variable bindings (parser::parse_and_visit_let_bindings in parse-statement-inl.h), reporting a different error than E0024 (error_let_with_no_bindings). I think E0036 is similar.

Hi, is this task closed or still available??? Thanks

[strager]

@Moshahrani You can work on it if you want.

[Moshahrani]

@Moshahrani You can work on it if you want.

Thanks!

[strager]

@Moshahrani Did you make progress? Do you need any help?

[matttheus]

I claim this for-hire task. I expect payment after I complete this task. I will email the quick-lint-js team if I am assigned this task.

[matttheus]

I'm still working on this issue

[matttheus]

Hey @strager I was not able to make progress on this issue, I will give up for now

[HicaroD]

Is this issue still opened by mistake?

I tried to simulate the bug in the QLJS playground:

[strager]

The bug is still present. Here's a repro:

function useState() {}

const [a, setA] = useState(),
const [b, setB] = useState();