LuaLS / LuaLS/lua-language-server

FR: Option that "Ignore this line" Code Actions adds to this line instead of the one above

Ouverte
#1,726 1 commentaire 9 réactions 0 personnes assignées Voir sur GitHub

Personne n'a encore pris cette issue.

enhancement
Langage dominant
Lua
Étoiles
4.4k
Forks
442
Métriques de merge des PR
Aucune PR mergée en 30 j

Description

Right now, when using the "Ignore this line" code action, it adds this line above the respective line:
```lua
---@diagnostic disable-next-line: rulename
problemline
```

However, I often prefer this type of line ignoring
```lua
problemline ---@diagnostic disable-line: rulename
```

while it works and is correctly recognized by the lsp, there is no code action and/or no option to add such a comment, I have to change this manually.

---

The eslint-lsp, for example, [has an option for the ignore-line-code-action which accepts `sameLine` or `separateLine`](https://github.com/microsoft/vscode-eslint/blob/55871979d7af184bf09af491b6ea35ebd56822cf/server/src/eslintServer.ts#L2119).

Guide de contribution

Ouvrir le guide de contribution

Par où commencer

  1. Lisez l'issue en entier, puis le guide de contribution du projet.
  2. Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
  3. Forkez le dépôt et travaillez sur une branche.
  4. Ouvrez une pull request qui référence le numéro de l'issue.

Piste de recherche

Commencez par localiser l’implémentation du language server de l’action de code « Ignore this line » et la manière dont ses paramètres sont définis. Comparez le comportement existant de disable-next-line avec la forme disable-line sur la même ligne demandée ; le travail est terminé lorsque les utilisateurs peuvent choisir l’emplacement via une option et que le commentaire Lua généré est reconnu correctement.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Évaluation

Stack technique
lua
Domaine
devtools
Type d'issue
Fonctionnalité
Difficulté
3/5
Temps estimé
1-2 jours
Activité
À l'abandon
Clarté
Plutôt claire
Accessibilité débutants
35/100

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.