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 ?
- 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
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