added sonarLint to the table and updated the RSLint Rules link

[Xanas1252]

updated the RSLint Rules link since the current one returns 404 page (done in a new PR)

• added SonarLint to the table except the speed and LSP server

[github-actions]

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).

[Xanas1252]

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

[strager]

Move this into a separate commit or PR.

[strager]

This should be "good" and should mention "free", like quick-lint-js' cell.

[strager]

Blocking: Add missing cell for LSP server.

It looks like their VS Code plugin uses an LSP server, so I'd say ✅ for SonarLint+LSP.

[strager]

Blocking: Add a cell for SonarLint for the speed column. Write "?" if you're unsure about the speed; we can fill it in later after we run benchmarks.

[strager]

The table looks really wide. We could use ­ to hyphenate "SonarLint" as "Sonar-Lint" to reduce the table width:

Suggested change

	<th><a href="https://sonarlint.org/">SonarLint</a></th>
	<th><a href="https://sonarlint.org/">Sonar&shy;Lint</a></th>

[strager]

Blocking: SonarLint's website claims it has an IntelliJ (aka WebStorm) plugin. Does it not work for JS code? https://www.sonarlint.org/intellij/

[strager]

Regarding the CLA:

It looks like GitHub's API isn't returning your comment at all:

$ curl \
  -H "Accept: application/vnd.github.v3+json" \
  https://api.github.com/repos/quick-lint/quick-lint-js/pulls/355/comments | jq 'map(.body)'
[
  "Move this into a separate commit or PR.",
  "This should be \"good\" and should mention \"free\", like quick-lint-js' cell.",
  "Blocking: Add missing cell for LSP server.\r\n\r\nIt looks like their VS Code plugin uses an LSP server, so I'd say ✅ for SonarLint+LSP.",
  "Blocking: Add a cell for SonarLint for the speed column. Write \"?\" if you're unsure about the speed; we can fill it in later after we run benchmarks.",
  "The table looks really wide. We could use `&shy;` to hyphenate \"SonarLint\" as \"Sonar-Lint\" to reduce the table width:\r\n```suggestion\r\n              <th><a href=\"https://sonarlint.org/\">Sonar&shy;Lint</a></th>\r\n```",
  "Blocking: SonarLint's website claims it has an IntelliJ (aka WebStorm) plugin. Does it not work for JS code? https://www.sonarlint.org/intellij/"
]

$ curl \
  -H "Accept: application/vnd.github.v3+json" \
  https://api.github.com/repos/quick-lint/quick-lint-js/issues/355/comments | jq 'map(.body)'
[
  "**CLA Assistant Lite bot:** Thank you for your contribution! Like many free software projects, you must sign our [Contributor License Agreement](https://github.com/quick-lint/quick-lint-js/blob/master/docs/CLA-v1.md) before we can accept your contribution. You can agree to the Contributor License Agreement by copy-pasting the following message into a new comment on this GitHub pull request:\n   - - -\n   ***I have read and hereby agree to quick-lint-js' Contributor License Agreement (CLA-v1.md).***\n   - - -\n   <sub>You can retrigger this bot by commenting **recheck** in this Pull Request</sub>"
]

Maybe this is why the CLA bot isn't detecting your comment.

@Xanas1252 Can you try reposting the CLA acceptance message in a new comment?

[Xanas1252]

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

[strager]

@Xanas1252 do you plan to update this PR?