microsoft / microsoft/TypeScript
Preserve JSDocs in *.d.ts files when stripping comments
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- Go
- Sterne
- 111k
- Forks
- 14.3k
- Ø Merge
- 1 T. 19 Std.
- Gemergte PRs (30 T.)
- 117
Beschreibung
Suggestion/Feature
When using the switch removeComments it will strip out all comments including the ones you may consider useful to the end consumer such as JSDocs. I'd like to strip out any internal comments but preserve any JSDocs comments. It seems that an option to preserve comments in types would be sufficient.
Existing functionality
"removeComments": true,
"declaration": true,
removes all comments including those in *.d.ts files.
Suggested functionality
"removeComments": true,
"declaration": true,
I would think by default you'd want to include the JSDocs in your type defs, so this would remove all comments from the code, but retain any comments associated with the types ie JSDocs.
If however, there is a need to remove all comments then an additional switch may be required to serve that purpose:
"removeCommentsIncludingDeclarations": true,
"declaration": true,
Would produce the same result we have today.
Beitragsleitfaden
Erste Schritte
- Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
- Forke das Repository und arbeite in einem Branch.
- Öffne einen Pull Request, der die Issue-Nummer nennt.
Rechercherichtung
Reproduziere zunächst das Problem mit aktivierten Optionen "removeComments" und "declaration" und vergleiche anschließend die erzeugte JavaScript- und .d.ts-Ausgabe. Die Aufgabe ist abgeschlossen, wenn interne Kommentare entfernt werden, während mit Deklarationen verknüpfte JSDoc-Kommentare erhalten bleiben, und es eine dokumentierte Möglichkeit gibt, bei Bedarf das aktuelle Verhalten zum Entfernen aller Kommentare beizubehalten.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- typescript
- Bereich
- compilers
- Issue-Typ
- Feature
- Schwierigkeit
- 4/5
- Geschätzter Aufwand
- 3-5 Tage
- Aktivitätsstatus
- Veraltet
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 45/100