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 ?

Đang mở
#2,301 0 bình luận 0 reaction 0 người được giao Xem trên GitHub
Ngôn ngữ chính
C++
Star
2.1k
Fork
503
Chỉ số merge pull request
Không có pull request nào được merge trong 30 ngày

Mô tả

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

Hướng dẫn đóng góp

Mở hướng dẫn đóng góp

Hướng nghiên cứu

Bắt đầu với src/apps/ociocheck/main.cpp quanh dòng 90 và lần theo cách Interop IDs và namespaces được kiểm tra. Xác nhận hành vi dự kiến đối với một ID bắt đầu bằng ':' và mức độ cảnh báo dự kiến khi một ID có namespace khớp với một ID chính thức. Được coi là hoàn tất khi hành vi và cách diễn đạt đã được thống nhất và được các kiểm tra ociocheck liên quan bao phủ.

Do mô hình lập chỉ mục viết ra từ nội dung của issue.

Đánh giá

Công nghệ
cpp
Lĩnh vực
cli
Loại issue
Lỗi
Độ khó
2/5
Thời gian dự kiến
1-3 giờ
Mức độ hoạt động
Ít trao đổi
Độ rõ ràng
Khá rõ ràng
Mức phù hợp với người mới
45/100

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.