bregctl check: a duplicate record type apiName is reported on the other record type, not on the authored apiName

Aperta Adatta ai principianti
  • Qualcosa non va?
#2,381 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub

I maintainer di solito rispondono entro 1 giorno

Nessuno ha ancora preso questa issue.

Valutazione

Difficoltà
2/5
Tempo stimato
1-3 ore
Idoneità per principianti
85/100
Tipo di issue
Bug
Chiarezza
Specificata chiaramente
Stato di attività
Attiva
Stack tecnologico
rust
Ambito
cli

Direzione di ricerca

In crates/registry-breg/src/compiler.rs, trovare validate_native_definitions dove viene sollevato l'errore breg.record-type.api-name-invalid. Modificarlo in modo che punti al apiName scritto su directory-entry (riga 31 dell'esempio) e che nel messaggio indichi il nome duplicato e l'altro record type. Fatto quando eseguendo bregctl --format json check sul repro l'errore viene riportato alla riga scritta con la formulazione attesa.

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

Descrizione

Follow-up to #1940 (case 4b), measured on a build of #2355's head; compiler.rs and check.rs are unchanged at main 85cd3f7d.

Repro: in products/breg/examples/facility, add apiName: Operator to the directory-entry record type (line 31), then run bregctl --format json check.

What happens: one error, breg.record-type.api-name-invalid, at /recordTypes/1, line 41 (- id: operator), with the message "Record type API names must be unique upper camel case names (in recordTypes[].apiName)". The operator record type has no apiName; its API name is derived from its id. validate_native_definitions in crates/registry-breg/src/compiler.rs reports whichever record type it reaches second.

Expected: the error points at the authored apiName: Operator line on directory-entry, and the message names the duplicated name and the other record type, for example "API name Operator on directory-entry is already used by operator (derived from its id)". When both names are authored, pointing at the second one is fine, as long as the message names the first.

The list path from #1940 is gone and the suggestedAction is specific; only the placement and the message are left.

Lingua principale
Rust
Stelle
2
Fork
0
Merge medio
9h
PR unite (30g)
200

Preparare l'ambiente

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 registrystack/registry-stack

Tutte le issue di registrystack/registry-stack

Issue simili

Altre issue su Rust

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.