audit-translations stops on the new empty gender lists

Đang mở Phù hợp với người mới
  • Có gì không ổn?
#994 0 bình luận 0 reaction 0 người được giao Xem trên GitHub

Maintainer thường phản hồi trong vòng 1 ngày

Chưa có ai nhận issue này.

Đánh giá

Độ khó
1/5
Thời gian dự kiến
Dưới một giờ
Mức phù hợp với người mới
88/100
Loại issue
Lỗi
Độ rõ ràng
Đặc tả rõ ràng
Mức độ hoạt động
Sôi nổi
Công nghệ
python, rust
Lĩnh vực
internationalization, tooling

Hướng nghiên cứu

Đọc PythonScripts/audit_translations/parsers.py và so sánh cách xử lý các ánh xạ rỗng với src/definitions.rs. Trước tiên, chạy kiểm tra bản dịch tiếng Tây Ban Nha để xác nhận lỗi đã được báo cáo. Công việc hoàn tất khi các ánh xạ rỗng được xử lý như các tập hợp rỗng và kiểm tra tiếng Tây Ban Nha hoàn thành mà không có quy tắc thiếu mới.

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

Mô tả

Since the unit gender change, audit-translations stops for es, pt, fr and de:

✗ Error: Invalid definition 'MasculineCurrencyWords' in Rules/Languages/es/definitions.yaml at line 742: empty mappings have no unambiguous definition kind

(fr and de stop at FeminineUnitWords.) These lists are empty on purpose, and src/definitions.rs reads an empty mapping as an empty set. Doing the same in PythonScripts/audit_translations/parsers.py fixes it:

     if isinstance(value, dict):
         if not value:
-            raise invalid("empty mappings have no unambiguous definition kind")
+            return DefinitionKind.SET  # same as src/definitions.rs: an empty mapping is an empty set

With this change the Spanish audit runs again (no new missing rules).

Ngôn ngữ chính
Rust
Star
131
Fork
97
Merge trung bình
21 giờ 45 phút
Pull request đã merge (30 ngày)
95

Chuẩn bị môi trường

Bắt đầu từ đâu

  1. Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
  2. Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
  3. Fork repository và làm thay đổi trên một nhánh.
  4. Mở pull request có tham chiếu số hiệu của issue.

Issue khác của daisy/MathCAT

Tất cả issue của daisy/MathCAT

Issue tương tự

Thêm issue về Rust

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.