glideapps/quicktype

Superfluous C# using statements

Open

#800 ouverte le 15 avr. 2018

Voir sur GitHub
 (0 commentaires) (0 réactions) (0 assignés)TypeScript (10 867 stars) (968 forks)batch import
C#good first issue

Description

The using statements introduced for DateTimeOffset support (if I recall correctly) are emitted in attributes-only mode, and when there are no dates in the model:

using System.Globalization;
using Newtonsoft.Json.Converters;

Guide contributeur

Stack technique
typescriptcsharp
Domaine
toolingdeveloper experience
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.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
under 1 hour
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
stale
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
clear
Prérequis
C# syntaxquicktype codebase
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
70
Direction de recherche
Examine the C# code generation logic in quicktype, likely in src/language/csharp.ts or similar. Look for where using statements are emitted, particularly for DateTimeOffset and Newtonsoft.Json. Ensure they are only emitted when needed (e.g., when dates are present or not in attributes only mode).