AcademySoftwareFoundation / AcademySoftwareFoundation/OpenColorIO

NIT PICKS ociocheck: Empty namespace for Interop ID? Warning message too strong when a valid Interop ID is used with a namespace ?

Offen
#2,301 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen
Vorherrschende Sprache
C++
Sterne
2.1k
Forks
503
PR-Merge-Kennzahlen
Keine gemergten PRs in 30 T.

Beschreibung

Whilst investigating the unused variable [here](https://github.com/AcademySoftwareFoundation/OpenColorIO/blob/main/src/apps/ociocheck/main.cpp#L90) I had some questions:

1. I wondered what it should do if the namespace is empty? (If the string begins with a ':')

2. I note that ociocheck will warn if a namespaced ID has the same spelling as an official ID - all good but do we feel the word 'must' is a little strong here? I know we are not a "Standard" body, but I'm imagining a world where a pre-official ID is being migrated to an official one and a user of ociocheck now thinks the file is in error. We'd obviously want users to migrate to an official one so maybe switch to 'should' ?

Beitragsleitfaden

Beitragsleitfaden öffnen

Rechercherichtung

Beginne mit src/apps/ociocheck/main.cpp ungefähr bei Zeile 90 und verfolge, wie Interop IDs und Namespaces geprüft werden. Bestätige das erwartete Verhalten für eine ID, die mit ':' beginnt, sowie die beabsichtigte Stärke der Warnung, wenn eine ID mit Namespace mit einer offiziellen ID übereinstimmt. Als erledigt gilt die Aufgabe, wenn Verhalten und Formulierung festgelegt und durch die relevanten ociocheck-Prüfungen abgedeckt sind.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
cpp
Bereich
cli
Issue-Typ
Bug
Schwierigkeit
2/5
Geschätzter Aufwand
1-3 Stunden
Aktivitätsstatus
Ruhig
Klarheit
Größtenteils klar
Anfängerfreundlichkeit
45/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.