Microsoft/monaco-editor

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

Open

#3468 aperta il 18 dic 2022

Vedi su GitHub
 (0 commenti) (0 reazioni) (0 assegnatari)JavaScript (1283 fork)batch import
editor-corefeature-requesthelp wanted

Metriche repository

Star
 (14.836 star)
Metriche merge PR
 (Merge medio 6h 26m) (15 PR mergiate in 30 g)

Descrizione

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.

Guida contributor

Direzione di ricerca
Individua la definizione dell'interfaccia Command nel codice sorgente di monaco editor, probabilmente in un file come 'src/vs/editor/common/languages.ts'. Modifica la proprietà 'id' da 'string' a 'string | undefined'. Verifica che la modifica venga compilata e che il comportamento dell'etichetta cliccabile rimanga come previsto. Controlla se ci sono altri utilizzi che dipendono dal fatto che 'id' sia obbligatorio.
Tech stack
typescript
Dominio
api
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
1
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Meno di 1 ora
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Nuova
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
TypeScriptGit
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
85