microsoft / microsoft/TypeScript

Harden response delivery to ensure services is consistent with server

Offen
#44,260 1 Kommentar 0 Reaktionen 1 zugewiesene Person Auf GitHub ansehen

@orta arbeitet bereits daran.

Seit 07.6.2021.

Domain: LS: TSServer Infrastructure Rescheduled
Vorherrschende Sprache
Go
Sterne
111k
Forks
14.4k
Ø Merge
1 T. 19 Std.
Gemergte PRs (30 T.)
117

Beschreibung

Spoke with @andrewbranch

If you [add] a member to any protocol.*Response, there's a significant chance that your fourslash tests [will] work but it [won't] actually work over the TS Server because session.ts picked apart the response in the process of converting positions to line/character and didn't know to put it back

How could we fix it?

Destructuring with an object rest instead of requiring session.ts to name every property it wants to preserve

Beitragsleitfaden

Beitragsleitfaden öffnen

Erste Schritte

  1. Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
  3. Forke das Repository und arbeite in einem Branch.
  4. Öffne einen Pull Request, der die Issue-Nummer nennt.

Bewertung

Dieses Issue wurde noch nicht bewertet.

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.