Fix few issues into release-candidate branch, with now the branch not contains

Đang mở
  • Có gì không ổn?
#889 2 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ó
4/5
Thời gian dự kiến
3-5 ngày
Mức phù hợp với người mới
25/100
Loại issue
Lỗi
Độ rõ ràng
Khá rõ ràng
Mức độ hoạt động
Sôi nổi
Công nghệ
rust, yaml
Lĩnh vực
internationalization, testing

Hướng nghiên cứu

Review the three attached patches and compare their changes with the release-candidate and main branches. Start with Rules/Languages/en/unicode-full.yaml and Rules/Languages/hu/navigate.yaml, then inspect tests/languages/hu/navigate.rs and tests/Language/hu/mtable.rs. Confirm which changes fit the release-candidate policy and whether they preserve the reported Hungarian translation fixes and tests.

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

Mô tả

discussion rules translation

Hi Boys,

@NSoiffer, @moritz-gross, I doed three release-candidate branch compatible patches, with do all wanted fixes to release-candidate branch contains the latest good hungarian translations, similar with main branch.
I fixed following things:

  • Fixing unwanted duplicates with Rules/Languages/en/unicode-full.yaml file,
  • Added october 6 merged Rules/Languages/hu/navigate.yaml change (without test code files, if the test file code modifications is not fitting the release-candidate branch policy),
  • Create a patch with applying tests/languages/hu/navigate.rs new test, similar with main branch, and add new row separator and column separator related tests in tests/Language/hu/mtable.rs file.
    Because me not have possibility to open a PR with release-candidate branch target branch, I attaching both three patches. Please merge selective this changes to the release-candidate branch, depending with the release-candidate branch policy.
    I don't no when coming out the next MathCAT stable release (I seeing the 0.7.7 release is comed out yesterday, congratulate the new release).
    I would like only to land this new october merged translation fixes are landing the new stable release in future. :-):-)
    Me full good if you merging only the two yaml files related patches to the release-candidate branch, if Rust test files code modification or new test file code file creation is not enabled into RC state.

I attaching the three patches:
fix_unwanted_unicode_duplicates.patch
fix_navigate_translations.patch

add_navigate_and_mtable_row_separator_tests.patch

Hopefully any I this day doed change are you merging the upstream release-candidate branch. Me translator audit tool now shows good results, similar with main branch, and all 637 hungarian tests are right passed my local system, similar with main branch.
So, if you merging for example the two yaml files related two patches, the hungarian main branch awailable MathCAT translations and release-candidate branch translations are equals good, equals the october 6 fixed state.

I wish for you a good weekend, and very congratulate the new MathCAT 7.7 stable release.

Attila

Ngôn ngữ chính
Rust
Star
130
Fork
96
Merge trung bình
1 ngày 1 giờ
Pull request đã merge (30 ngày)
84

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.