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 ?
- Langage dominant
- C++
- Étoiles
- 2.1k
- Forks
- 503
- Métriques de merge des PR
- Aucune PR mergée en 30 j
Description
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' ?
Guide de contribution
Ouvrir le guide de contribution
Piste de recherche
Commencez par src/apps/ociocheck/main.cpp vers la ligne 90 et suivez la manière dont les Interop IDs et les namespaces sont vérifiés. Confirmez le comportement attendu pour un ID commençant par ':' ainsi que le niveau de sévérité prévu pour l’avertissement lorsqu’un ID avec namespace correspond à un ID officiel. Le travail est terminé lorsque le comportement et la formulation sont définis et couverts par les vérifications ociocheck pertinentes.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Évaluation
- Stack technique
- cpp
- Domaine
- cli
- Type d'issue
- Bug
- Difficulté
- 2/5
- Temps estimé
- 1-3 heures
- Activité
- Calme
- Clarté
- Plutôt claire
- Accessibilité débutants
- 45/100