feat: new diag for using ‘.’ after ‘?.’

quick-lint / quick-lint-js

quick-lint-js finds bugs in JavaScript programs

https://quick-lint-js.com

GNU General Public License v3.0

1.54k stars 192 forks source link

feat: new diag for using ‘.’ after ‘?.’ #1133

Closed pepplejoshua closed 9 months ago

pepplejoshua commented 10 months ago

This closes #1128 -- it warns the user about using a dot After an optional chain. This cover optional chain with function call as well as optional chain with index expressions

github-actions[bot] commented 10 months ago

CLA Assistant Lite bot Thank you for your contribution! Like many free software projects, you must sign our Contributor License Agreement before we can accept your contribution.

EDIT: All contributors have signed quick-lint-js' Contributor License Agreement (CLA-v1.md).

pepplejoshua commented 10 months ago

I have read and hereby agree to quick-lint-js' Contributor License Agreement (CLA-v1.md).

strager commented 9 months ago

Landed as commit bdfccbdc8c405c5f0d9d40ce8c6736d9c31a6a4a. Thanks for the patch!