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 ?

Ouverte
#2,301 0 commentaires 0 réactions 0 personnes assignées Voir sur GitHub
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

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.