Retire liblcm refusals once FieldWorks pins a liblcm with #410/#413/#414

Aperta
  • Qualcosa non va?
#159 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à
3/5
Tempo stimato
1-2 giorni
Idoneità per principianti
48/100
Tipo di issue
Refactoring
Chiarezza
Specificata chiaramente
Stato di attività
Attiva
Stack tecnologico
csharp
Ambito
backend

Direzione di ricerca

Read LiblcmRefusals, LiblcmRefusalSweepTests and UnstorablePropertyTests, then check the FieldWorks SilLcmVersion pin and whether a liblcm release includes fixes #410, #413 and #414. After that release is pinned, rerun the sweep and reduce the refusal rules to cases liblcm still throws, keeping DroppedProperty/OnDropped for remaining refusals. The issue says control characters still need handling under liblcm#415.

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

Descrizione

SIL.TextEditing.LCModel refuses text properties and values that liblcm 11.0.0-beta0180 throws on (LiblcmRefusals, reported through OnDropped). The liblcm fixes sillsdev/liblcm#410, #413 and #414 remove almost all of those throws.

Evidence, from a local build of the three PRs combined (-p:SilLcmVersion=11.0.0-stack410-413-414.gb9b80fa, packed into the local feed):

  • Nothing is refused by property number any more: 50, 129, 150–155, 159 and 160 all load.
  • Most integer range rules and the string rules for properties 1–5, 7, 130, 133, 145 and 146 are unnecessary: 7457 integer and 9178 string cases are "liblcm loads, rules refuse".
  • There are 0 cases of "liblcm throws, rules accept".
  • 9 refusal tests fail as expected: the 5 in LiblcmRefusalSweepTests and 4 in UnstorablePropertyTests.

When FieldWorks pins a liblcm release with those fixes:

  1. Bump SilLcmVersion.
  2. Rerun the sweep and shrink LiblcmRefusals to what still throws. The sweep is the single source of truth.
  3. Keep DroppedProperty/OnDropped for anything left. Control characters are still deleted on save; see sillsdev/liblcm#415.

Blocked on a liblcm release and the FieldWorks pin.

Lingua principale
C#
Stelle
0
Fork
0
Merge medio
1h 43m
PR unite (30g)
43

Preparare l'ambiente

Questo progetto non fornisce container di sviluppo, Dockerfile né guida per i contributori, quindi l'ambiente è a tuo carico: parti dal suo README e consulta la nostra guida al primo contributo per i passaggi generali.

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 sillsdev/SIL.TextEditing

Tutte le issue di sillsdev/SIL.TextEditing

Issue simili

Altre issue su C#

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.