Microsoft/monaco-editor

[Bug] monaco.languages.Command.id should be typed as optional

Open

#3 468 ouverte le 18 déc. 2022

Voir sur GitHub
 (0 commentaires) (0 réactions) (0 assignés)JavaScript (1 283 forks)batch import
editor-corefeature-requesthelp wanted

Métriques du dépôt

Stars
 (14 836 stars)
Métriques de merge PR
 (Merge moyen 6h 26m) (15 PRs mergées en 30 j)

Description

Reproducible in vscode.dev or in VS Code Desktop?

  • Not reproducible in vscode.dev or VS Code Desktop

Reproducible in the monaco editor playground?

Monaco Editor Playground Code

// https://microsoft.github.io/monaco-editor/playground.html#extending-language-services-codelens-provider-example

Reproduction Steps

Experiment with removing id: commandId, and notice the label is no longer clickable

Actual (Problematic) Behavior

Whether or not an id is present in the code lens command controls whether or not the label is clickable or not. Typing indicate that id is required, however: https://microsoft.github.io/monaco-editor/api/interfaces/monaco.languages.Command.html.

Expected Behavior

monaco.languages.Command.id should be id?: string or id: string | undefined.

Additional Context

This came up on CompilerExplorer (https://godbolt.org) when we converted a js file to ts.

Guide contributeur

Direction de recherche
Localisez la définition de l'interface Command dans le code source de monaco editor, probablement dans un fichier tel que 'src/vs/editor/common/languages.ts'. Modifiez la propriété 'id' de 'string' à 'string | undefined'. Vérifiez que la modification compile et que le comportement du libellé cliquable reste comme prévu. Vérifiez s'il existe d'autres utilisations qui dépendent du fait que 'id' soit obligatoire.
Stack technique
typescript
Domaine
api
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
1
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Moins d'une heure
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Récente
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
TypeScriptGit
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
85