Julia template: `eval` of `parse` anti-pattern

Aperta Adatta ai principianti
#23,941 2 commenti 0 reazioni 0 assegnatari Vedi su GitHub

Nessuno ha ancora preso questa issue.

Valutazione

Difficoltà
2/5
Tempo stimato
1-3 ore
Idoneità per principianti
78/100
Tipo di issue
Refactoring
Chiarezza
Specificata chiaramente
Stato di attività
Tranquilla
Stack tecnologico
julia
Ambito
tooling

Direzione di ricerca

Inizia dalle righe del resolver indicate in modules/openapi-generator/src/main/resources/julia-server/partial_model_oneof.mustache e julia-client/partial_model_oneof.mustache. Verifica che le sostituzioni dei nomi dei modelli siano identificatori Julia validi e preservino il comportamento del resolver generato. Il lavoro è completato quando entrambi i template usano riferimenti diretti ai modelli senza eval né Meta.parse.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Descrizione

The julia-server and julia-client generators emit the OpenAPI.property_type discriminator resolver using eval(Base.Meta.parse("ModelName")). The model types are already in scope in the generated module, so straight ModelName works: ~30× faster on the resolver call and pre-compilation friendly, while keeping the behavior the same.

Relevant template pieces: lines 16, 18 here and here: simply replace with return {{modelName}} and return {{.}}.

Worth confirming {{modelName}} is always a valid Julia identifier (true for normal model names) before merging.

Lingua principale
Java
Stelle
26.8k
Fork
7.7k
Merge medio
1g 10h
PR unite (30g)
103

Guida per i contributori

Apri la guida per i contributori

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Altre issue di OpenAPITools/openapi-generator

Tutte le issue di OpenAPITools/openapi-generator

Issue simili

Altre issue su Java

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.